Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Exclude dependabot from community-contribution labeling #101742

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

Merged
merged 3 commits into from
Jul 30, 2024

Conversation

jeffhandley
Copy link
Member

As seen in recent pull requests generated by dependabot, we are applying the community-contribution Indicates that the PR has been added by a community member label to PRs generated by dependabot. This adds an exclusion to the automation to prevent that label from being applied.

It's not 100% clear if the [bot] suffix is needed on the name, and we have inconsistency in this config and across other repos. I'm adding the suffix here because I see label-prs.yml (dotnet/docs) does not have the suffix and dependabot PRs are gaining the label. Once confirmed that the suffix works as expected here, I'll follow up with a PR to the docs repo to adjust that policy too.

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Apr 30, 2024
@jeffhandley jeffhandley added area-Meta and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Apr 30, 2024
@jeffhandley
Copy link
Member Author

The YmlValidation check seems to be failing on existing configuration values. I will investigate later today.

@jeffhandley jeffhandley added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label May 3, 2024
@akoeplinger
Copy link
Member

I'm pretty sure the [bot] suffix is needed since that is what the API considers as the user login, e.g. from https://api.github.com/repos/dotnet/runtime/pulls/101585

  "user": {
        "login": "dependabot[bot]",
        "id": 49699333,

@jeffhandley jeffhandley removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jul 30, 2024
@jeffhandley jeffhandley requested a review from akoeplinger July 30, 2024 02:30
@jeffhandley jeffhandley merged commit c92fe4b into main Jul 30, 2024
13 checks passed
@jeffhandley jeffhandley deleted the jeffhandley/dependabot-pr-labeling branch July 30, 2024 22:02
@github-actions github-actions bot locked and limited conversation to collaborators Aug 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants