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-ignore #62

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 (1dba4f4) 76.74% compared to head (a57c2e1) 80.18%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
+ Coverage   76.74%   80.18%   +3.44%     
==========================================
  Files           3        3              
  Lines         258      212      -46     
==========================================
- Hits          198      170      -28     
+ Misses         45       27      -18     
  Partials       15       15              
Flag Coverage Δ
oldstable 80.18% <ø> (+3.44%) ⬆️
stable 80.18% <ø> (+3.44%) ⬆️

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 8176155 into go-openapi:master Feb 3, 2024
11 checks passed
@fredbi fredbi deleted the fix/ci-exclude-paths branch February 3, 2024 17:28
# 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