-
Notifications
You must be signed in to change notification settings - Fork 30
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
Modify build_gh_pages.sh in order to create zipped maps for pipeline … #2915
Modify build_gh_pages.sh in order to create zipped maps for pipeline … #2915
Conversation
@@ -22,12 +22,16 @@ $CCSH sonarimport -nc -o codecharta_sonar.cc.json https://sonarcloud.io maibornw | |||
$CCSH modify --set-root root/maibornwolff-gmbh_codecharta_visualization -o codecharta_sonar_mod.cc.json codecharta_sonar.cc.json | |||
|
|||
$CCSH merge -o ../visualization/app/codecharta.cc.json codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json -nc | |||
# Zipped map for pipeline build | |||
$CCSH merge -o ../visualization/app/codecharta.cc.json codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json |
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.
$CCSH merge -o ../visualization/app/codecharta.cc.json codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json | |
$CCSH merge -o ../visualization/app/codecharta.cc.json.gz codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json |
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.
I tried your suggestion and we don't need it. The merge filter puts .gz at the end automatically.
$CCSH modify --set-root root/maibornwolff-gmbh_codecharta_analysis -o codecharta_sonar_mod.cc.json codecharta_sonar_analysis.cc.json | ||
$CCSH modify --set-root root/analysis -o codecharta_git_mod.cc.json codecharta_git.cc.json | ||
$CCSH merge -o ../visualization/app/codecharta_analysis.cc.json codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json -nc | ||
# Zipped map for pipeline build | ||
$CCSH merge -o ../visualization/app/codecharta_analysis.cc.json codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json |
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.
$CCSH merge -o ../visualization/app/codecharta_analysis.cc.json codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json | |
$CCSH merge -o ../visualization/app/codecharta_analysis.cc.json.gz codecharta_sonar_mod.cc.json codecharta_git_mod.cc.json |
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.
I tried your suggestion and we don't need it. The merge filter puts .gz
at the end automatically.
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.
Did zippedMap.cc.json.gz was added intentionally?
Let's add a changelog entry, so that the users know the reason for the upcoming patch release.
Yes, we did this for testing purposes, as a second approach to read zipped files. |
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] Kudos, SonarCloud Quality Gate passed! |
Create zipped maps in pipeline build
Issue: #2538
We would like to test this modification with a patch release.