SONARPY-2432: Split build into building and testing #2236
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Seppli11
force-pushed
the
SONARPY-2431
branch
from
40df6ad to
e17e5d5
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
from
ba5586f to
ddecf09
Compare
Seppli11
force-pushed
the
SONARPY-2431
branch
3 times, most recently
from
a85f87c to
747a97a
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
from
ddecf09 to
c7b1a0c
Compare
Seppli11
force-pushed
the
SONARPY-2431
branch
from
747a97a to
f73c181
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
from
c7b1a0c to
acecf15
Compare
.cirrus.yml
Outdated
| @@ -94,10 +94,29 @@ build_task: | |||
| - git checkout | |||
| - cd - | |||
| - source cirrus-env BUILD | |||
| - regular_mvn_build_deploy_analyze -DfailStubGenerationFast=true | |||
| - regular_mvn_build_deploy_analyze -DfailStubGenerationFast=true -DskipTypshed -DskipTests=true -Dsonar.skip=true | |||
There was a problem hiding this comment.
If we're not executing the tests, could we even go one step further and not initialize the submodules?
Or do we still expect things to fail here in case of checksum mismatch?
We'll probably be able to remove the Next token here I guess?
There was a problem hiding this comment.
I think you're right, we don't need it here. I moved it down to the analyze_task, where we probably do need it.
Seppli11
force-pushed
the
SONARPY-2432
branch
4 times, most recently
from
bd6e15d to
2aa229a
Compare
Seppli11
force-pushed
the
SONARPY-2431
branch
from
f73c181 to
d71aaa7
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
5 times, most recently
from
31758b7 to
ec54ceb
Compare
Seppli11
force-pushed
the
SONARPY-2431
branch
from
d71aaa7 to
101b8aa
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
3 times, most recently
from
6e1d504 to
5be5f0f
Compare
Base automatically changed from
SONARPY-2431
to
ruling-parallelization-master
December 12, 2024 12:56
Seppli11
force-pushed
the
SONARPY-2432
branch
2 times, most recently
from
86e49c0 to
84cf586
Compare
Seppli11
force-pushed
the
ruling-parallelization-master
branch
from
ef81724 to
2535acf
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
2 times, most recently
from
c5aa790 to
c0217e0
Compare
Seppli11
force-pushed
the
ruling-parallelization-master
branch
from
2535acf to
919136b
Compare
Seppli11
force-pushed
the
SONARPY-2432
branch
from
c0217e0 to
2b7ef65
Compare
guillaume-dequenne-sonarsource
approved these changes
Dec 13, 2024
Seppli11
force-pushed
the
SONARPY-2432
branch
from
2b7ef65 to
c7cb88b
Compare
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SONARPY-2432