feat(ci): migrate sonar analysis, nightly from Drone to GitHub Actions#297
feat(ci): migrate sonar analysis, nightly from Drone to GitHub Actions#297Ashim-Stha merged 1 commit intomainfrom
sonar analysis, nightly from Drone to GitHub Actions#297Conversation
Ashim-Stha
force-pushed
the
sonarAnalysis
branch
3 times, most recently
from
486d293 to
066aa4a
Compare
Ashim-Stha
force-pushed
the
sonarAnalysis
branch
13 times, most recently
from
eb3fe2e to
6b6d394
Compare
Ashim-Stha
changed the title
sonar analysis, nightly from Drone to GitHub Actions
Ashim-Stha
force-pushed
the
sonarAnalysis
branch
8 times, most recently
from
d848b33 to
69fb66c
Compare
Ashim-Stha
requested review from
PrajwolAmatya,
individual-it,
nirajacharya2,
phil-davis and
saw-jan
| runs-on: ubuntu-latest | ||
| needs: [php-unit-tests, php-integration-tests] | ||
| steps: | ||
| - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 |
There was a problem hiding this comment.
I guess the checkout is not needed ....
There was a problem hiding this comment.
I think the checkout is needed as it requires source code too and also the file sonar-project.properties
There was a problem hiding this comment.
we do not need to checkout for sonar-analysis
so I thought, you can check if this is true by removing the checkout and see what happens with the sonarcloud step. Include the CI link to that and we will have a visual proof for the future refs
There was a problem hiding this comment.
got this error after removing the checkout https://github.com/owncloud/ocis-php-sdk/actions/runs/24819358696/job/72640782693
There was a problem hiding this comment.
05:58:19.800 INFO Loaded core extensions: a3s, architecture, sca
05:58:20.062 INFO Process project properties
05:58:20.112 ERROR You must define the following mandatory properties for 'Unknown': sonar.projectKey, sonar.organization
05:58:20.438 INFO EXECUTION FAILURE
05:58:20.439 INFO Total time: 12.548s
Error: Action failed: The process '/opt/hostedtoolcache/sonar-scanner-cli/8.0.1.6346/linux-x64/bin/sonar-scanner' failed with exit code 3
Ashim-Stha
force-pushed
the
sonarAnalysis
branch
from
69fb66c to
c27c209
Compare
…ions Signed-off-by: Ashim Shrestha <ashimshrestha2384@gmail.com>
Ashim-Stha
force-pushed
the
sonarAnalysis
branch
from
c27c209 to
790f07f
Compare
|
DeepDiver1975
left a comment
DeepDiver1975
left a comment
There was a problem hiding this comment.
as far as I can tell ... 👍
3 participants
Description
Migrates the following Drone CI pipelines to GitHub Actions:
sonarAnalysisnightlyRelated Issue