Skip to content

Conversation

@Ablu
Copy link

@Ablu Ablu commented Oct 5, 2023

This should group updates into a single PR. Hopefully, that simplfies updates.

Updates that do actually require code changes, will need a separate PR anyway, after which dependabot can be asked to rebase/recreate.

Suggested-by: Patrick Roy [email protected]
Suggested-by: Stefano Garzarella [email protected]

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR are signed (with git commit -s), and the commit
    message has max 60 characters for the summary and max 75 characters for each
    description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

@Ablu
Copy link
Author

Ablu commented Oct 5, 2023

Discussed on Slack (https://rust-vmm.slack.com/archives/CFH3R8PFV/p1696517827753059), did not put it into the commit since the link will become invalid once the chat rotates out of our free Slack plan.

Copy link
Member

@stefano-garzarella stefano-garzarella left a comment

Choose a reason for hiding this comment

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

I was really looking forward to this feature! Thanks again @roypat for letting us discover it!

This should group updates into a single PR. Hopefully, that simplfies
updates.

Updates that do _actually_ require code changes, will need a separate PR
anyway, after which dependabot can be asked to rebase/recreate.

Suggested-by: Patrick Roy <[email protected]>
Suggested-by: Stefano Garzarella <[email protected]>
Signed-off-by: Erik Schilling <[email protected]>
@Ablu Ablu force-pushed the dependabot-grouping branch from acdcfe6 to 9f21b39 Compare October 5, 2023 15:54
@vireshk vireshk merged commit 072fada into rust-vmm:main Oct 6, 2023
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