-
Notifications
You must be signed in to change notification settings - Fork 16
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
Implemented new feature: display last error log in scenario description #24
Implemented new feature: display last error log in scenario description #24
Conversation
vrymar
commented
Mar 4, 2024
- Implemented new feature: display last error log in scenario description
- Implemented unit tests for the new feature
- Improved dependencies references in build.gradle
Implemented unit tests for the new feature Improved dependencies references in build.gradle
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #24 +/- ##
=============================================
+ Coverage 89.10% 89.71% +0.60%
- Complexity 128 131 +3
=============================================
Files 4 4
Lines 413 418 +5
Branches 37 35 -2
=============================================
+ Hits 368 375 +7
Misses 26 26
+ Partials 19 17 -2 ☔ View full report in Codecov by Sentry. |
…log-to-scenario-description # Conflicts: # build.gradle
…log-to-scenario-description # Conflicts: # src/main/java/com/epam/reportportal/karate/ReportPortalHook.java
@HardNorth could you please review this PR? |
@vrymar Sure, will do this tomorrow. |
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.
Good! Thanks for the work, just a few minor issues.
CHANGELOG.md
Outdated
@@ -4,6 +4,13 @@ | |||
### Removed | |||
- Shutdown hook register on supplied Launch, by @HardNorth | |||
|
|||
## [5.0.4] |
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.
Please don't add version tag, it is added automatically.
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.
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.
@vrymar Yes, exactly.
CHANGELOG.md
Outdated
### Removed | ||
- Shutdown hook register on supplied Launch, by @HardNorth |
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.
Please don't revert my changes.