Skip to content

ci: update what checks are required for PRs#550

Merged
RebeccaStevens merged 2 commits into
mainfrom
update-ci-requirements
Nov 21, 2024
Merged

ci: update what checks are required for PRs#550
RebeccaStevens merged 2 commits into
mainfrom
update-ci-requirements

Conversation

@RebeccaStevens

Copy link
Copy Markdown
Collaborator

fix #549

@RebeccaStevens

RebeccaStevens commented Nov 20, 2024

Copy link
Copy Markdown
Collaborator Author

@JoshuaKGoldberg could you update these settings manually in the repo settings? Then #547 should automerge. This PR can then be merged after that. I'll then rebase #531 and #535 off the latest main (#535's CI should then start passing after that).

Edit: I was able to bypass requiring you to make those changes by running the workflow manually.

@RebeccaStevens RebeccaStevens changed the title ci: update what checks are required for PRs ci: update what checks that are required for PRs Nov 20, 2024
@codecov

codecov Bot commented Nov 20, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.88%. Comparing base (c87c47b) to head (0a89871).
Report is 16 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #550      +/-   ##
==========================================
+ Coverage   80.59%   80.88%   +0.29%     
==========================================
  Files          35       35              
  Lines        6575     6655      +80     
  Branches      440      448       +8     
==========================================
+ Hits         5299     5383      +84     
+ Misses       1276     1272       -4     
Flag Coverage Δ
4.3.5 79.65% <ø> (-0.52%) ⬇️
4.6.4 79.68% <ø> (-0.52%) ⬇️
4.9.5 79.68% <ø> (-0.52%) ⬇️
latest 80.48% <ø> (+0.20%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@RebeccaStevens RebeccaStevens added the status: waiting for action Some action was requested and needs to be taken before things can continue label Nov 20, 2024
@RebeccaStevens RebeccaStevens force-pushed the update-ci-requirements branch 3 times, most recently from 0855898 to 3022316 Compare November 21, 2024 09:32
@RebeccaStevens RebeccaStevens removed the status: waiting for action Some action was requested and needs to be taken before things can continue label Nov 21, 2024
@RebeccaStevens RebeccaStevens merged commit 1336add into main Nov 21, 2024
@RebeccaStevens RebeccaStevens changed the title ci: update what checks that are required for PRs ci: update what checks are required for PRs Nov 21, 2024
@RebeccaStevens RebeccaStevens deleted the update-ci-requirements branch November 21, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🛠 Tooling: PR required checks, rely on renamed tasks

1 participant