Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: remove paths-ignores #87

Merged
merged 1 commit into from
Feb 3, 2024
Merged

Conversation

fredbi
Copy link
Member

@fredbi fredbi commented Feb 3, 2024

Using paths-ignore as a short-circuit for doc-only PRs does not work: branch protection rules cannot be passed.

Using paths-ignore as a short-circuit for doc-only PRs does not work:
branch protection rules cannot be passed.

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Copy link

codecov bot commented Feb 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a05020e) 87.16% compared to head (cd56a28) 88.04%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
+ Coverage   87.16%   88.04%   +0.87%     
==========================================
  Files          13       13              
  Lines        1839     1547     -292     
==========================================
- Hits         1603     1362     -241     
+ Misses        204      153      -51     
  Partials       32       32              
Flag Coverage Δ
oldstable 88.04% <ø> (+0.87%) ⬆️
stable 88.04% <ø> (+0.87%) ⬆️

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.

@fredbi fredbi merged commit db91acc into go-openapi:master Feb 3, 2024
11 checks passed
@fredbi fredbi deleted the fix/ci-exclude-paths branch February 3, 2024 18:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant