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

chore(ci): remove paths-ignore from CI workflow #572

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

jharley
Copy link
Collaborator

@jharley jharley commented Nov 15, 2024

While "optimal" to have these in place, they end up meaning that the CI workflow doesn't run but it is required for merge -- preventing non-repo-Admins from merging release PRs (mostly notably).


@jharley jharley added the no-changelog Skip the changelog label Nov 15, 2024
@jharley jharley requested a review from a team as a code owner November 15, 2024 14:46
@codecov-commenter
Copy link

codecov-commenter commented Nov 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.55%. Comparing base (d58baef) to head (f67e062).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #572   +/-   ##
=======================================
  Coverage   75.55%   75.55%           
=======================================
  Files          87       87           
  Lines        7286     7286           
=======================================
  Hits         5505     5505           
  Misses       1429     1429           
  Partials      352      352           

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

@jharley jharley merged commit f8111b9 into main Nov 15, 2024
6 checks passed
@jharley jharley deleted the jharley.ci-adjust-ignorechanges branch November 15, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Skip the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants