Skip to content

Test BUILD-10215: standardize GitHub Actions output logging#121

Closed
bwalsh434 wants to merge 1 commit intomasterfrom
test-BUILD-10215-standardize-ghaction-output-logging
Closed

Test BUILD-10215: standardize GitHub Actions output logging#121
bwalsh434 wants to merge 1 commit intomasterfrom
test-BUILD-10215-standardize-ghaction-output-logging

Conversation

@bwalsh434
Copy link
Copy Markdown

Summary

  • Point ci-github-actions references to BUILD-10215-standardize-ghaction-output-logging branch for testing
  • Tests the standardization of GitHub Actions output logging (::error and ::warning annotations)

Test plan

  • Verify build workflow runs successfully
  • Check that error/warning annotations appear correctly in the GitHub Actions UI

Note: This is a temporary test PR. Revert to @master after testing.

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 3, 2026 23:36
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR temporarily redirects the repository’s Build workflow to use a test branch of the shared SonarSource/ci-github-actions actions to validate standardized GitHub Actions output logging/annotations.

Changes:

  • Point build-npm action usage to BUILD-10215-standardize-ghaction-output-logging
  • Point config-npm action usage to BUILD-10215-standardize-ghaction-output-logging
  • Point promote action usage to BUILD-10215-standardize-ghaction-output-logging

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- name: Build, Test, Analyze and deploy
id: build
uses: SonarSource/ci-github-actions/build-npm@master # dogfood
uses: SonarSource/ci-github-actions/build-npm@BUILD-10215-standardize-ghaction-output-logging # dogfood
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow now references a feature branch (@BUILD-10215-standardize-ghaction-output-logging). If this PR is merged and that branch is renamed/deleted, the build job will break. Before merging, switch back to a stable ref (e.g., @master as used for other dogfood workflows in this repo, or pin to an immutable commit SHA/tag).

Copilot uses AI. Check for mistakes.
with:
version: 2025.12.13
- uses: SonarSource/ci-github-actions/config-npm@master # dogfood
- uses: SonarSource/ci-github-actions/config-npm@BUILD-10215-standardize-ghaction-output-logging # dogfood
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This step uses SonarSource/ci-github-actions/config-npm@BUILD-10215-standardize-ghaction-output-logging, which is a mutable branch reference. To avoid CI breakage/supply-chain risk on merge, revert to the stable ref used elsewhere (e.g., @master) or pin to an immutable commit SHA/tag before merging.

Suggested change
- uses: SonarSource/ci-github-actions/config-npm@BUILD-10215-standardize-ghaction-output-logging # dogfood
- uses: SonarSource/ci-github-actions/config-npm@master

Copilot uses AI. Check for mistakes.
BUILD_NUMBER: ${{ needs.build.outputs.BUILD_NUMBER }} # reuse BUILD_NUMBER from build job
steps:
- uses: SonarSource/ci-github-actions/promote@master # dogfood
- uses: SonarSource/ci-github-actions/promote@BUILD-10215-standardize-ghaction-output-logging # dogfood
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

promote@BUILD-10215-standardize-ghaction-output-logging is a temporary branch ref. If merged, future runs may fail once the branch is cleaned up. Please revert to @master (per existing dogfood usage) or pin to an immutable SHA/tag before merging.

Copilot uses AI. Check for mistakes.
@sonarqube-next
Copy link
Copy Markdown

sonarqube-next bot commented Mar 4, 2026

Quality Gate passed Quality Gate passed for 'sonar-dummy-js'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@bwalsh434 bwalsh434 closed this Mar 9, 2026
@bwalsh434 bwalsh434 deleted the test-BUILD-10215-standardize-ghaction-output-logging branch March 9, 2026 18:56
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.

2 participants