Skip to content

Add just recipes for updating vendored dependencies#388

Merged
rebkwok merged 4 commits intomainfrom
vendored-dependencies
Mar 18, 2026
Merged

Add just recipes for updating vendored dependencies#388
rebkwok merged 4 commits intomainfrom
vendored-dependencies

Conversation

@rebkwok
Copy link
Copy Markdown
Contributor

@rebkwok rebkwok commented Mar 18, 2026

  • add a copy of the upgrade-pipeline script used in other report
  • move update vendored dependencies from script to a just recipe
  • add recipe for updating pipeline which also updates vendored dependencies

Copy link
Copy Markdown
Member

@bloodearnest bloodearnest left a comment

Choose a reason for hiding this comment

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

This makes sense to me, thanks.

It used to be more complex, and having it as an external script enabled shellcheck for it. But its pretty simple now, so I think embedding it as as just recipe makes sense.

The main reason I wrote update-pipeline.sh as separate script was to enable easily updating it across multiple repos.

But perhaps it could also be an embedded recipe?

@rebkwok
Copy link
Copy Markdown
Contributor Author

rebkwok commented Mar 18, 2026

But perhaps it could also be an embedded recipe?

I think it could, but I'll maybe leave that until we do it across all of them at once

@rebkwok rebkwok merged commit e63d4c4 into main Mar 18, 2026
15 checks passed
@rebkwok rebkwok deleted the vendored-dependencies branch March 18, 2026 13:28
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