Skip to content

Remove checks on pinned versions#4579

Merged
pfcoperez merged 4 commits into
mainfrom
remove/check_pinned_versions
Jul 14, 2026
Merged

Remove checks on pinned versions#4579
pfcoperez merged 4 commits into
mainfrom
remove/check_pinned_versions

Conversation

@pfcoperez

Copy link
Copy Markdown
Member

Our dependencies updates come:

  • Mostly from Renovate through automatic updates.
  • When we want to remove a pinned version, through a manual PR removing the pin (less frequently with two remaining).

We aim to not having pinned versions and if we do instruct Renovate to stop updating them.
We also have some precedents of pinned packages that are just declared in Renovate: #4547

After this PR, will trust Renovate and go.mod as unique sources of truth for pins with tests as barriers.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can remove the entire skill I think, renovate has been doing a good job

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was very close to delete it completely, 100% on removing it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in 92b1400
🎉 Determinism!

@pfcoperez
pfcoperez marked this pull request as ready for review July 14, 2026 17:30
@pfcoperez
pfcoperez requested a review from a team as a code owner July 14, 2026 17:30
@claude

claude Bot commented Jul 14, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@pfcoperez
pfcoperez enabled auto-merge (squash) July 14, 2026 17:52
@pfcoperez
pfcoperez merged commit ad1b62e into main Jul 14, 2026
23 checks passed
@pfcoperez
pfcoperez deleted the remove/check_pinned_versions branch July 14, 2026 17:53
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.

2 participants