Skip to content

Add sonar cloud testing to CI#2187

Merged
tylerjw merged 5 commits intomainfrom
sonarcloud
May 19, 2023
Merged

Add sonar cloud testing to CI#2187
tylerjw merged 5 commits intomainfrom
sonarcloud

Conversation

@tylerjw
Copy link
Copy Markdown
Member

@tylerjw tylerjw commented May 18, 2023

Description

This is for testing using SonarCloud static analysis in our CI.

@codecov
Copy link
Copy Markdown

codecov bot commented May 18, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.01 🎉

Comparison is base (895ecac) 50.54% compared to head (daa1463) 50.55%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2187      +/-   ##
==========================================
+ Coverage   50.54%   50.55%   +0.01%     
==========================================
  Files         387      387              
  Lines       31719    31719              
==========================================
+ Hits        16028    16031       +3     
+ Misses      15691    15688       -3     

see 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sonarqubecloud
Copy link
Copy Markdown

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

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

Copy link
Copy Markdown
Member

@EzraBrooks EzraBrooks left a comment

Choose a reason for hiding this comment

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

This LGTM; not reporting anything but I think that may be because we haven't scanned main yet.

@tylerjw tylerjw merged commit 243b0b2 into main May 19, 2023
@tylerjw tylerjw deleted the sonarcloud branch May 19, 2023 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants