Skip to content

Conversation

@senekor
Copy link
Contributor

@senekor senekor commented Dec 4, 2025

@senekor senekor requested a review from a team as a code owner December 4, 2025 22:53
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Hello 👋 Thanks for your PR.

This repo does not currently have dedicated maintainers. Our guardians team will attempt to review and merge your PR, but it will likely take longer for your PR to be reviewed.

If you enjoy contributing to Exercism and have a track-record of doing so successfully, you might like to become an Exercism maintainer for this track.

Please feel free to ask any questions, or chat to us about anything to do with this PR or the reviewing process on the Exercism forum.

(cc @exercism/guardians)

@IsaacG
Copy link
Member

IsaacG commented Dec 5, 2025

Would pinning pin it to the minor/patch version? Is that a problem?

@senekor
Copy link
Contributor Author

senekor commented Dec 5, 2025

So, the "pinning" is just a comment on the "important" libraries. If these are updated in a semver-breaking way (1.0 -> 2.0 or also 0.1 -> 0.2), I hope the comment will cause humans to read the longer instructions above. Non-semver-breaking updates can still be applied via the lockfile by running cargo update.

@IsaacG
Copy link
Member

IsaacG commented Dec 5, 2025

Gotcha. So this won't block dependabot from creating PRs that attempt to bump them?

@senekor
Copy link
Contributor Author

senekor commented Dec 5, 2025

That's right, dependabot doesn't read these comments. There would be specific syntax to make automated tools aware of the pinning. I think something like this:

time = "=0.3.0"

But that's not the goal, we do want to offer newer versions when they come out. We just want a reminder for humans to also include the old version, to avoid breaking lots of exercise solutions.

@senekor senekor merged commit 44f9c68 into main Dec 5, 2025
3 checks passed
@senekor senekor deleted the senekor/zxnvqzqztnkm branch December 5, 2025 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants