Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[vividus] Stop logging debug messages from soft assert into allure appender #1611

Merged
merged 1 commit into from
May 5, 2021
Merged

[vividus] Stop logging debug messages from soft assert into allure appender #1611

merged 1 commit into from
May 5, 2021

Conversation

uarlouski
Copy link
Member

@uarlouski uarlouski commented May 5, 2021 •
edited by valfirst
Loading

Closes #1593

@sonarcloud
Copy link

sonarcloud bot commented May 5, 2021

Kudos, SonarCloud Quality Gate passed!

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov
Copy link

codecov bot commented May 5, 2021

Codecov Report

Merging #1611 (92f0115) into master (3f126db) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1611   +/-   ##
=========================================
  Coverage     95.76%   95.76%           
  Complexity     5094     5094           
=========================================
  Files           712      712           
  Lines         14622    14622           
  Branches        999      999           
=========================================
  Hits          14003    14003           
  Misses          485      485           
  Partials        134      134

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 3f126db...92f0115. Read the comment docs.

@valfirst valfirst changed the title [vividus] Stop logging debug messages from soft assert into allure ap… [vividus] Stop logging debug messages from soft assert into allure appender May 5, 2021
@valfirst valfirst merged commit 7311e44 into master May 5, 2021
@valfirst valfirst deleted the removeDebug branch May 5, 2021 11:37
@valfirst valfirst mentioned this pull request Jun 14, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@valfirst valfirst valfirst approved these changes

@ikalinin1 ikalinin1 ikalinin1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allure report: Duplicated success message
3 participants
@uarlouski @valfirst @ikalinin1