Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unignore Cargo.lock file #3104

Merged
merged 1 commit into from
Dec 26, 2024
Merged

Unignore Cargo.lock file #3104

merged 1 commit into from
Dec 26, 2024

Conversation

Turbo87
Copy link
Collaborator

@Turbo87 Turbo87 commented Dec 26, 2024

This makes it much easier to checkout a previous revision of the project and have a working set of dependencies. That helps for example with things like running git bisect when finding regressions.

see also https://blog.rust-lang.org/2023/08/29/committing-lockfiles.html

This makes it much easier to checkout a previous revision of the project and have a working set of dependencies. That helps for example with things like running `git bisect` when finding regressions.

see also https://blog.rust-lang.org/2023/08/29/committing-lockfiles.html
@Turbo87
Copy link
Collaborator Author

Turbo87 commented Dec 26, 2024

I would recommend to also use https://github.com/renovatebot/renovate to regularly update the lockfile (and probably also dev-dependencies), but that will need to be installed in the tokio-rs GitHub org to be able to use it and idk if any of our axum maintainers have sufficient permissions to do so 🤔

@Turbo87
Copy link
Collaborator Author

Turbo87 commented Dec 26, 2024

@jplatte for future PRs: are you watching the repo (i.e. do you get notifications for new PRs anyway?) or would you prefer an explicit ping or review request?

@jplatte
Copy link
Member

jplatte commented Dec 26, 2024

Makes sense to me.

Re. renovate: We could set up dependabot instead, this doesn't require any special permissions. It seems to support quite many modes of operation. How about a weekly PR? I'd prefer every other week but unfortunately it seems like they only support a small fixed number of cadences.

Re. review: I get notified of everything already, no need to ping or request review.

@Turbo87
Copy link
Collaborator Author

Turbo87 commented Dec 26, 2024

Re. renovate: We could set up dependabot instead, this doesn't require any special permissions. It seems to support quite many modes of operation.

I used to recommend dependabot over renovatebot a few years ago, but have changed my mind since then. renovatebot is way more flexible and configurable. the only advantage of dependabot is that it was bought by GitHub and integrated somewhat, but the original creator isn't even working for GitHub anymore...

How about a weekly PR? I'd prefer every other week but unfortunately it seems like they only support a small fixed number of cadences.

The default cadence for lockfile updates in renovatebot is weekly, which works well in my experience.

In renovatebot the schedule is essentially a cronjob syntax (see e.g. https://docs.renovatebot.com/presets-schedule/#scheduleautomergemonthly) which could be adjusted to run on fixed days 2-3 times per month, if we wanted to go that way.

@Turbo87 Turbo87 merged commit ee7544d into tokio-rs:main Dec 26, 2024
18 checks passed
@Turbo87 Turbo87 deleted the lockfile branch December 26, 2024 13:36
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