Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rp #2300

Merged
merged 2 commits into from
Dec 21, 2021
Merged

Rp #2300

merged 2 commits into from
Dec 21, 2021

Conversation

ikalinin1
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Dec 17, 2021

Codecov Report

Merging #2300 (8093197) into master (b18a067) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 8093197 differs from pull request most recent head 78aac9d. Consider uploading reports for the commit 78aac9d to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2300      +/-   ##
============================================
+ Coverage     88.00%   88.03%   +0.02%     
- Complexity     5544     5554      +10     
============================================
  Files           762      762              
  Lines         15403    15439      +36     
  Branches       1033     1033              
============================================
+ Hits          13556    13592      +36     
  Misses         1712     1712              
  Partials        135      135              
Impacted Files Coverage Δ
...idus/reportportal/listener/AttachmentListener.java 100.00% <ø> (ø)
...reportportal/config/ReportPortalConfiguration.java 100.00% <100.00%> (ø)
...ve/AdaptedDelegatingReportPortalStoryReporter.java 100.00% <100.00%> (ø)
...eportportal/jbehave/AdaptedReportPortalFormat.java 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b18a067...78aac9d. Read the comment docs.

implementation project(':vividus-soft-assert')
implementation project(':vividus-reporter')
implementation(group: 'com.google.guava', name: 'guava', version: versions.guava)
implementation(group: 'org.springframework', name: 'spring-context', version: versions.spring)

// ReportPortal
implementation(group: 'com.epam.reportportal', name: 'logger-java-log4j', version: '4.0.2')
implementation(group: 'com.github.reportportal', name: 'agent-java-jbehave', version: '4.2.1') {
implementation(group: 'com.epam.reportportal', name: 'logger-java-log4j', version: '5.1.2')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
implementation(group: 'com.epam.reportportal', name: 'logger-java-log4j', version: '5.1.2')
implementation(group: 'com.epam.reportportal', name: 'logger-java-log4j', version: '5.1.3')

welcome to log4jhell

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@valfirst valfirst merged commit f309982 into master Dec 21, 2021
@valfirst valfirst deleted the rp branch December 21, 2021 11:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants