-
Notifications
You must be signed in to change notification settings - Fork 25
ci: remove sonarcloud CI configs #1804
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
Conversation
79585e5 to
494d703
Compare
|
I think we miss |
yeah, need sonarcloud setup. what's the plan? remove sonarcloud configs for now? |
Yes please. Currently there are no plans for setting up SonarCloud. |
494d703 to
cd94435
Compare
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
cd94435 to
b1dc6fe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes sonarcloud CI configurations and unnecessary Node.js installation commands from the Woodpecker CI pipeline configuration.
Changes:
- Removed commented-out sonarcloud configuration and environment variables from the unitTests function
- Removed temporary Node.js 22 installation commands from the translation_sync function
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
b1dc6fe to
345bc7d
Compare
* ci: remove sonarcloud configs Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com> * ci: remove unnecessary commands Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com> --------- Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Description
Remove sonarcloud CI configs and some unnecessary commands.
Related Issue
How Has This Been Tested?
Types of changes