-
Notifications
You must be signed in to change notification settings - Fork 62
feat: add noAuthors
option
#183
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
Conversation
test: add test for skipAuthors option test: add test commits fixture
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Co-authored-by: Philipp Kief <philipp.kief@gmx.de>
@PKief don't understand why but all the tests pass locally but not in the |
@kleinpetr it is a snapshot mismatch, please don't worry about it will check before merge. And again sorry this PR got delayed this much I had been busy on so many other projects. |
Something looks a little weird, because the CI expects this line to be part of the snapshot: Locally, it's working fine for me as well. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #183 +/- ##
===========================================
- Coverage 69.10% 45.55% -23.55%
===========================================
Files 9 12 +3
Lines 916 957 +41
Branches 85 102 +17
===========================================
- Hits 633 436 -197
- Misses 283 518 +235
- Partials 0 3 +3 β View full report in Codecov by Sentry. |
noAuthors
option
Resolves #174
β Type of change
π Description
the new option
--skipAuthors
to avoid Contributors sectionπ Checklist