Skip to content

BadCommitSummary: version check should be ignored for acct-* packages#434

Merged
arthurzam merged 1 commit intopkgcore:masterfrom
arthurzam:fix-BadCommitSummary
Aug 19, 2022
Merged

BadCommitSummary: version check should be ignored for acct-* packages#434
arthurzam merged 1 commit intopkgcore:masterfrom
arthurzam:fix-BadCommitSummary

Conversation

@arthurzam
Copy link
Copy Markdown
Member

When doing the checks for commit summary, add an exception to the rule
for acct-* packages, so the version added isn't a must for them.

Resolves: #433

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 17, 2022

Codecov Report

Base: 95.97% // Head: 95.97% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (40ef7e9) compared to base (8d8d956).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #434   +/-   ##
=======================================
  Coverage   95.97%   95.97%           
=======================================
  Files          55       55           
  Lines        7833     7834    +1     
  Branches     2218     2218           
=======================================
+ Hits         7518     7519    +1     
  Misses        194      194           
  Partials      121      121           
Impacted Files Coverage Δ
src/pkgcheck/checks/git.py 97.62% <100.00%> (+<0.01%) ⬆️
src/pkgcheck/checks/metadata.py 98.11% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@arthurzam arthurzam force-pushed the fix-BadCommitSummary branch from 98d3aa0 to 1ffe9a2 Compare August 17, 2022 19:27
When doing the checks for commit summary, add an exception to the rule
for acct-* packages, so the version added isn't a must for them.

Resolves: pkgcore#433
Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
@arthurzam arthurzam force-pushed the fix-BadCommitSummary branch from 1ffe9a2 to 40ef7e9 Compare August 19, 2022 18:07
@arthurzam arthurzam merged commit 40ef7e9 into pkgcore:master Aug 19, 2022
@arthurzam arthurzam deleted the fix-BadCommitSummary branch August 19, 2022 18:24
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.

"BadCommitSummary" version check should be ignored for acct-* packages

1 participant