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

ci(workflows): add PR title scope check #829

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

pinglin
Copy link
Member

@pinglin pinglin commented Nov 10, 2024

Because

  • the codebase now includes additional components, it’s better for PRs to mention the specific scope.

This commit

  • add PR title scope requirement.

Copy link

codecov bot commented Nov 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 20.08%. Comparing base (7e5d3de) to head (14eab68).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #829      +/-   ##
==========================================
+ Coverage   20.01%   20.08%   +0.06%     
==========================================
  Files         354      355       +1     
  Lines       74750    74912     +162     
==========================================
+ Hits        14963    15046      +83     
- Misses      57571    57642      +71     
- Partials     2216     2224       +8     
Flag Coverage Δ
unittests 20.08% <ø> (+0.06%) ⬆️

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.

@donch1989 donch1989 merged commit 38d2216 into main Nov 11, 2024
14 checks passed
@donch1989 donch1989 deleted the pinglin/ci-add-pr-scope-check branch November 11, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants