Skip to content

Conversation

@reneleonhardt
Copy link
Contributor

Chores

  • Update cargo and npm dependencies
  • Let dependabot update cargo, github-actions, npm

@reneleonhardt
Copy link
Contributor Author

There is no CI for pull requests, please check manually.

@stegaBOB
Copy link
Contributor

@Swatinem Would be great to get this merged in! I think there are some fixes to caching issues with warp builds that this update pulls in

@reneleonhardt reneleonhardt force-pushed the chore/update-dependencies branch from 563a65e to 1d54079 Compare July 25, 2025 20:25
@reneleonhardt reneleonhardt force-pushed the chore/update-dependencies branch from 1d54079 to 7c68b1a Compare July 25, 2025 20:45
Copy link
Owner

Choose a reason for hiding this comment

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

I appreciate your consideration to add this.
However, so far I have always had a bad experience with dependabot and its bad defaults of opening tens of tiny PRs for every single patch version.

So I would appreciate if we either remove this, or tweak the config in such a way that it is not as spammy and annoying as the default settings.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added an example configuration, you can fine-tune it to your needs now.
But I recommend to wait for eventual "spamming" and improve the configuration then, better use that time to improve CI and test coverage until then.

@Swatinem Swatinem merged commit a8b36e3 into Swatinem:master Jul 26, 2025
23 checks passed
@reneleonhardt reneleonhardt deleted the chore/update-dependencies branch July 26, 2025 19:12
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