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

fix: exclude tests from coverage #83

Merged
merged 1 commit into from
Apr 16, 2024
Merged

fix: exclude tests from coverage #83

merged 1 commit into from
Apr 16, 2024

Conversation

Aidasvenc
Copy link
Collaborator

What I did

excluded tests folder from the Sonar coverage calculation.

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor

@Gaellosada Gaellosada left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Gaellosada Gaellosada left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@GaelCondeLosada GaelCondeLosada left a comment

Choose a reason for hiding this comment

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

LGTM

@Aidasvenc Aidasvenc merged commit 1f8911b into main Apr 16, 2024
3 checks passed
Aidasvenc added a commit that referenced this pull request Apr 16, 2024
# 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