Charlotte
Charlotte
> Thanks for the answers @svenseeberg and @charludo. I don't want to block the pull request further. βΊοΈ No worries. Having to merge this fast is a not-too-great situation to...
> LGTM, thanks π > > One thing I would change in the future is not showing internal comments as official responses. The red bordered message is also returned in...
@svenseeberg @timobrembeck @david-venhoff the three of you seemed most interested at the conference, would one of you mind doing the second review? π
> I also get an error  > > The env seems to contain these values though π€  Interesting. Not sure why that would happen, but I'll have a...
@david-venhoff [this SO post](https://stackoverflow.com/questions/50746147/postgresqls-initdb-fails-with-invalid-locale-settings-check-lang-and-lc-e) seemed relevant, describing the exact same issue with postgres. The first answer is some hacky workarounds.... ...and the second answer is "you didn't mention nix, but...
> I found [this](https://discourse.nixos.org/t/new-install-nix-wants-glibc-2-38-debian-12-has-2-36/36109/4) post which suggests to `unset LD_LIBRARY_PATH` and then the server finally starts on my laptop. I don't really know what this means though or where that...
> I didn't check out the other PR yet, but does it essentially tackle the same problems? If so, I'd suggest taking a vote in the team for the preferred...
Well said @deen13. One thing I'd like to add: I can see an maintainence-effort argument against keeping both solutions, but the maintainence required should be minimal, since in both cases...
> We would just also have to remove all the dev tools that depend on them. But since pre-commit can also run single checks, I don't see an important reason...
> Given how long it takes to get pull requests reviewed and merged, I hesitate to add more complexity here. Small moves. @PeterNerlich Yes, sorry, that was more of a...