Sage-Bionetworks / synapsePythonClient

Programmatic interface to Synapse services for Python
https://www.synapse.org
Apache License 2.0
65 stars 67 forks source link

[SYNPY-1470]Remove SonarCloud's dependency on test job #1094

Closed danlu1 closed 1 month ago

danlu1 commented 1 month ago

Problem:

Currently, SonarCloud only executes when tests pass, but even if tests fail, its valuable for sonarcloud to execute.

Solution:

Remove needs:[test] so it doesn't require the test job complete successfully before kicking off SonarCloud job. Add if: ${{ always() }} in the SonarCloud step so the it can be run at all time no matter the status of previous steps.