Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Allow dependabot handling to be configured #227

Merged
merged 2 commits into from
Aug 26, 2021

Conversation

skitt
Copy link
Contributor

@skitt skitt commented Aug 26, 2021

If run from a pull_request_target event, the provided token allows
writes even from dependabot PRs. To support this, without requiring
all users to use pull_request_target, this patch adds a
configuration option to ignore dependabot PRs.

The default, which matches the documented pull_request_target use,
is to handle dependabot PRs. Setting ignore_dependabot to on in
the configuration will skip dependabot PRs instead.

Fixes: #226
Signed-off-by: Stephen Kitt skitt@redhat.com

@skitt skitt force-pushed the configurable-dependabot branch from 6bfa2c2 to e81ff4b Compare August 26, 2021 10:03
If run from a `pull_request_target` event, the provided token allows
writes even from dependabot PRs. To support this, without requiring
all users to use `pull_request_target`, this patch adds a
configuration option to ignore dependabot PRs.

The default, which matches the documented `pull_request_target` use,
is to handle dependabot PRs. Setting `ignore_dependabot` to `on` in
the configuration will skip dependabot PRs instead.

Fixes: z0al#226
Signed-off-by: Stephen Kitt <skitt@redhat.com>
@skitt skitt force-pushed the configurable-dependabot branch from e81ff4b to ae5a6e1 Compare August 26, 2021 10:13
Copy link
Owner

@z0al z0al left a comment

Choose a reason for hiding this comment

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

Looks great @skitt 💯 . Thank you so much!

src/support.ts Outdated Show resolved Hide resolved
@z0al z0al merged commit eaef15f into z0al:main Aug 26, 2021
@skitt skitt deleted the configurable-dependabot branch August 26, 2021 12:02
@github-actions
Copy link

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dependabot issues are ignored, but don’t need to be
2 participants