-
Notifications
You must be signed in to change notification settings - Fork 994
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
Pass the serialized group name when creating a PR for grouped updates #7166
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of questions about removing the grouped_update
key from the payload ( which I think is fine ) and moving away from using the term id
.
Co-authored-by: Barry Gordon <896971+brrygrdn@users.noreply.github.com>
99bef01
to
5529ce8
Compare
Co-authored-by: Barry Gordon <896971+brrygrdn@users.noreply.github.com>
4e78793
to
f6916ee
Compare
I'll update the smoke-tests to use the new dependency-group key instead of grouped-update See dependabot/cli#102 for the CLI update that's required to update the smoke tests |
…d-update key Grouped updates now pass in the name of the group the PR was created with instead of a boolean grouped-update flag /xref dependabot/dependabot-core#7166
Opened dependabot/smoke-tests#61 for updating the smoke tests. The CLI changes need to be deployed first though |
…d-update key (#102) Grouped updates now pass in the name of the group the PR was created with instead of a boolean grouped-update flag /xref dependabot/dependabot-core#7166
…dency-group-id-to-api Pass the serialized group name when creating a PR for grouped updates
Pass the serialized group name to PR creator (and the API client) when creating a PR for grouped updates.
We no longer pass the
grouped-update
key to the PR creator and instead pass in thedependency-group
nameFor now we are transforming the group name to downcase, snake_case, and to a symbol (although the test checks the name after it's been serialized to JSON, so the DependencyChange#id is converted to a string)