Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Use lowercase "dependabot" name for mocked Dependabot github commits #52

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Use lowercase "dependabot" name for mocked Dependabot github commits #52

merged 1 commit into from
Sep 19, 2023

Conversation

rmartin16
Copy link
Member

@rmartin16 rmartin16 commented Sep 19, 2023 •
edited
Loading

Changes

  • When mocking commits for Dependabot, use "dependabot" as the name for the commit instead of "Dependabot"

For example, here's a recent PR from Dependabot:

| * f147742e - (2 days ago) Add changenote. [dependabot skip] - Dependabot[bot] (origin/dependabot/pip/sphinx-7.2.6)
| * 63792c1f - (2 days ago) Bump sphinx from 7.2.5 to 7.2.6 - dependabot[bot]

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

freakboy3742
freakboy3742 approved these changes Sep 19, 2023
View reviewed changes
Copy link
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

👍

@freakboy3742 freakboy3742 merged commit 40710f1 into beeware:main Sep 19, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@freakboy3742 freakboy3742 freakboy3742 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rmartin16 @freakboy3742