Skip to content
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

PR and issue automation #1370

Merged
merged 5 commits into from
Jun 30, 2023
Merged

Conversation

martincostello
Copy link
Member

  • Add the dependency review action.
  • Use the new polly-updater-bot app to apply the .NET SDK updates. This removes the need to close and re-open PRs raised by the workflow to get the CI to run.
  • Add a workflow to approve and auto-merge dependabot updates for a trusted set of dependencies.
  • Add a workflow to approve and auto-merge .NET SDK updates by the polly-updater-bot app.
  • Automatically mark issues and pull requests as stale if there is no activity for 60 days.
  • Automatically close stale issues and pull requests after 14 days.

Resolves #1167.

Sort the list of dependencies.
Add the dependency review action.
Use the new `polly-updater-bot` app to apply the .NET SDK updates.
- Automatically mark issues and pull requests as stale if there is no activity for 60 days.
- Automatically close stale issues and pull requests after 14 days.
- Add a workflow to approve and auto-merge dependabot updates for a trusted set of dependencies.
- Add a workflow to approve and auto-merge .NET SDK updates by the polly-updater-bot app.
@martincostello martincostello added the github_actions Pull requests that update GitHub Actions code label Jun 29, 2023
@codecov
Copy link

codecov bot commented Jun 29, 2023

Codecov Report

Merging #1370 (5cf8b26) into main (0f425ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1370   +/-   ##
=======================================
  Coverage   83.61%   83.61%           
=======================================
  Files         268      268           
  Lines        6403     6403           
  Branches     1004     1004           
=======================================
  Hits         5354     5354           
  Misses        840      840           
  Partials      209      209           
Flag Coverage Δ
linux 83.61% <ø> (ø)
macos 83.61% <ø> (ø)
windows 83.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Contributor

@martintmk martintmk left a comment

Choose a reason for hiding this comment

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

lgtm, but I am no expert in GitHub workflows so take this review with a grain of salt.

Copy link
Member

@joelhulen joelhulen left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@martincostello
Copy link
Member Author

Cool - thanks both. I'll merge this as-is, and we can tweak anything in a future PR.

@martincostello martincostello marked this pull request as ready for review June 30, 2023 15:26
@martincostello martincostello merged commit 5734f4b into App-vNext:main Jun 30, 2023
@martincostello martincostello deleted the pr-automation branch June 30, 2023 15:26
@martincostello
Copy link
Member Author

Looks like the updater bot is working: logs

Run peter-murray/workflow-application-token-action@8e1ba3bf161[9](https://github.com/App-vNext/Polly/actions/runs/5424515902/jobs/9863981472#step:3:10)726336414f[10](https://github.com/App-vNext/Polly/actions/runs/5424515902/jobs/9863981472#step:3:11)14e37f17fbadf1db
Found GitHub Application: Polly Updater Bot
Obtaining application installation for repository: App-vNext/Polly
Requesting limitation on GitHub Application permissions to only: {"contents":"write","pull_requests":"write"}
{"token":"***","expires_at":"2023-06-30T16:27:17Z","permissions":{"contents":"write","metadata":"read","pull_requests":"write"},"repository_selection":"selected"}
Successfully generated an access token for application.

@martincostello
Copy link
Member Author

And so does the reviewer 🥳

#1376 (review)

@joelhulen
Copy link
Member

Awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/build github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pull request automation?
3 participants