Kristoffer Carlsson
Kristoffer Carlsson
I think https://github.com/JuliaLang/julia/pull/53957 should solve that.
I don't like how a random package adding a dependency (without even loading it) can cause code to behave differently.
Seems like something is not good right now: ``` ERROR: LoadError: ArgumentError: embedded NULs are not allowed in C strings: "/cache/build/builder-amdci5-4/julialang/julia-master/\0+\x16,\xe1v\xe4\xf0\xb8.L=\xdaK3\x89\xf4StyledStrings ```
It's not the `mktempdir` call but the fact that this file https://github.com/JuliaLang/Pkg.jl/blob/8b2c0f32939042e5d77da05d2825de6ec56487e9/test/test_packages/ArtifactOverrideLoading/src/ArtifactOverrideLoading.jl now gets set as read-only it then gets developed https://github.com/JuliaLang/Pkg.jl/blob/8b2c0f32939042e5d77da05d2825de6ec56487e9/test/artifacts.jl#L697-L701 and then attempts to be `touch`ed: https://github.com/JuliaLang/Pkg.jl/blob/8b2c0f32939042e5d77da05d2825de6ec56487e9/test/artifacts.jl#L736 (which...
I think it should copy it somewhere and only then mutate it.
Yes, this should be merged so StyledStrings need to be updated
It should be moved out of Base (into StyledStrings or its own stdlib). `join` and `*` will have to be figured out later. This + https://github.com/JuliaLang/StyledStrings.jl/issues/61 + all other places...
> As alluded to in my last message, and stated on Slack, over the past ~month I simply haven't had the time for this. I think this highlights the danger...
Anyway, I don't understand why this is on the milestone when the existing API would still work if using NamedTuples instead of tuples so adding that will not break anything....
Ok, then we declare the whole thing experimental. This has been on the milestone for two months for some new color highlighting thingy. Insanity.