Skip to content

Conversation

@straight-shoota
Copy link
Member

Example PR: #16386

@ysbaddaden
Copy link
Collaborator

What's the point of devenv.lock if we keep updating it every week automatically?

@straight-shoota
Copy link
Member Author

It describes the current state and ensures consistency.
But since we're following a rolling release of nixpkgs, we can get regular updates.
I don't think there's any reason to prevent that?
We can talk about the interval, of course.

@ysbaddaden
Copy link
Collaborator

What's the benefit of following the rolling release of nixpkgs instead of the latest stable?

@ysbaddaden
Copy link
Collaborator

I mean, it sounds like mere noise and hassle to keep the environment updated. We could instead follow nixpkgs releases, that updates twice a year for example.

@straight-shoota
Copy link
Member Author

Then we'd be using older versions of the tools. I don't think this would be a good idea.

@straight-shoota
Copy link
Member Author

I'd rather have the validation hooks up-to-date than risk falling out of sync with the versions we use and miss critical validation.
Devenv itself is also actively developed and has relatively frequent new releases.

I'm happy to try anything else, though. Just felt it's a nuisance to do this maintenance job manually.

@ysbaddaden
Copy link
Collaborator

A 6 months span doesn't feel like we'd lag far behind to me bu 🤷

@straight-shoota
Copy link
Member Author

straight-shoota commented Nov 21, 2025

I've been waiting on an update for actionlint for a couple of weeks already. 😅
It doesn't recognize the new GitHub action runner value macos-15-intel yet, so pre-commit currently fails on macos.yml workflow: https://github.com/crystal-lang/crystal/actions/runs/19548833173/job/55974600067

@straight-shoota straight-shoota added this to the 1.19.0 milestone Nov 21, 2025
@straight-shoota straight-shoota merged commit aa3829f into master Nov 24, 2025
86 checks passed
@straight-shoota straight-shoota deleted the infra/devenv-update-ci branch November 24, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants