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

Fix some SonarCloud warnings. #883

Merged
merged 2 commits into from
Oct 11, 2022
Merged

Fix some SonarCloud warnings. #883

merged 2 commits into from
Oct 11, 2022

Conversation

TyVik
Copy link
Owner

@TyVik TyVik commented Oct 9, 2022

No description provided.

@TyVik TyVik self-assigned this Oct 9, 2022
@ghost
Copy link

ghost commented Oct 9, 2022 •
edited by ghost
Loading

👇 Click on the image for a new way to code review

  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@lgtm-com
Copy link

lgtm-com bot commented Oct 9, 2022

This pull request fixes 2 alerts when merging cc9e76d into f0181c7 - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class

@sonarcloud
Copy link

sonarcloud bot commented Oct 11, 2022

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 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@lgtm-com
Copy link

lgtm-com bot commented Oct 11, 2022

This pull request fixes 2 alerts when merging 86d1561 into f0181c7 - view on LGTM.com

fixed alerts:

  • 2 for Unused variable, import, function or class

@TyVik TyVik merged commit ac190b1 into develop Oct 11, 2022
@TyVik TyVik deleted the sonarcloud branch October 11, 2022 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@TyVik TyVik

Labels
Hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@TyVik