Skip to content

BUILD-7900 working with simple ci-common-scripts and complex Starlark pipelines#36

Merged
jayadeep-km-sonarsource merged 2 commits intomasterfrom
doc/jcarsique/BUILD-7900-cursor
Aug 6, 2025
Merged

BUILD-7900 working with simple ci-common-scripts and complex Starlark pipelines#36
jayadeep-km-sonarsource merged 2 commits intomasterfrom
doc/jcarsique/BUILD-7900-cursor

Conversation

@julien-carsique-sonarsource
Copy link
Copy Markdown
Contributor

@julien-carsique-sonarsource julien-carsique-sonarsource commented Aug 1, 2025

BUILD-7900

Fix naming for Cirrus CI
Remove extra verbosity.
Remove unnecessary cache_save thanks to pr-cleanup.yml.
Fix conditions and parameters.
Instruct to update documentation (README).
Replace pr-cleanup.yml if already present.
Remove/update Cirrus CI badges, path references, specifics...
Remove migrated tasks and files.
Help with understanding complex Starlark pipelines.

@julien-carsique-sonarsource julien-carsique-sonarsource force-pushed the doc/jcarsique/BUILD-7900-cursor branch from 4a2008f to 6dfb126 Compare August 1, 2025 17:36
@julien-carsique-sonarsource julien-carsique-sonarsource marked this pull request as ready for review August 1, 2025 17:36
@julien-carsique-sonarsource julien-carsique-sonarsource requested a review from a team August 1, 2025 17:36
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 6, 2025

🤖 Pull Request summary

Updates migration documentation to improve consistency and add critical documentation requirements.

Standardizes terminology: Changes "CirrusCI" to "Cirrus CI" throughout for consistency
Simplifies workflow examples: Removes unnecessary name fields and parameters in common action blocks
Adds critical documentation phase: New Phase 6 covering badge updates, path migrations, and workflow replacements
Enhances PR cleanup guidance: Distinguishes between manual implementations to replace vs. official SonarSource actions
Expands troubleshooting: Adds cirrus.generated.yml section for understanding cirrus-modules configurations

Key areas for review focus:

  • The new Phase 6 documentation requirements are comprehensive but may need validation against actual migration scenarios
  • PR cleanup detection logic assumes manual implementations are always 64+ lines - verify this heuristic
  • The cirrus.generated.yml generation command sequence should be

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@jayadeep-km-sonarsource jayadeep-km-sonarsource merged commit b868007 into master Aug 6, 2025
9 checks passed
@jayadeep-km-sonarsource jayadeep-km-sonarsource deleted the doc/jcarsique/BUILD-7900-cursor branch August 6, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants