-
Notifications
You must be signed in to change notification settings - Fork 2
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
85 #99
85 #99
Conversation
added implementation; tests
@h1alexbel take a look, please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@l3r8yJ thanks for you contribution! I have a few comments, take a look, please
src/test/java/git/tracehub/codereview/action/SkipIfMentionedTest.java
Outdated
Show resolved
Hide resolved
src/main/java/git/tracehub/codereview/action/SkipIfMentioned.java
Outdated
Show resolved
Hide resolved
@h1alexbel fixed, take a look, please |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #99 +/- ##
============================================
+ Coverage 59.76% 60.70% +0.93%
- Complexity 50 53 +3
============================================
Files 26 27 +1
Lines 251 257 +6
Branches 5 6 +1
============================================
+ Hits 150 156 +6
Misses 98 98
Partials 3 3 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@l3r8yJ one comment
@h1alexbel added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@l3r8yJ lgtm
@l3r8yJ thanks! |
closes #85
PR-Codex overview
This PR adds license headers to files, introduces a
PullRequestExtension
for testing, and implementsSkipIfMentioned
to skip PRs based on authors.Detailed summary
PullRequestExtension
for testingSkipIfMentioned
to skip PRs based on authors