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.

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

ci: remove sonar check #1030

Merged
merged 1 commit into from
Apr 27, 2023
Merged

ci: remove sonar check #1030

merged 1 commit into from
Apr 27, 2023

Conversation

algomaster99
Copy link
Member

@algomaster99 algomaster99 commented Apr 27, 2023

Removing Sonar check because it is throwing 500 in our CI job - https://github.com/ASSERT-KTH/sorald/actions/runs/4819525681/jobs/8583103332?pr=1029.

I never really bothered to check it so maybe it is okay to remove it. I have the plugin locally installed so it informs me of the sonar violations. If Sonar is fixed soon, I will integrate this back.

@algomaster99 algomaster99 merged commit 11f7622 into master Apr 27, 2023
@algomaster99 algomaster99 deleted the remove-sonar-check branch April 27, 2023 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant