Conversation
# Conflicts: # .cirrus.yml
Separate nightly analysis jobs from main build workflow to improve clarity: - build.yml now only runs SonarCloud analysis on every build - nightly.yml runs all platform analyses (SonarCloud, NEXT, SQC-US) and IRIS sync on schedule 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
… steps Replace the failing build-npm@v1 action with config-npm@v1 and manual npm commands for better control and to avoid template parsing issues. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Skip config-npm action and manually configure npm registry with vault credentials to avoid version field requirement. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Use correct vault path format with dashes instead of slashes: repository_owner-repository_name-private-reader 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Use correct vault path format for qa-deployer role 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Cache node_modules folder to avoid reinstalling dependencies in downstream jobs 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Install dependencies in test/integration and tools/orchestrator before running tests 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add cd .. commands to properly return to working directory after each step on Windows 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove shell: cmd specification to use PowerShell default - Split commands for better error handling - Use PowerShell's Copy-Item cmdlet - Add npm install step before package installation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This ensures consistent line endings across Windows and Linux, preventing prettier check-format failures on Windows runners. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…jest Using the full path with forward slashes doesn't work on Windows PowerShell. Just using 'jest' works cross-platform as npm adds node_modules/.bin to PATH. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Use bash shell on Windows (matching Cirrus behavior) so both QA jobs can use the same setup commands with subshells and forward slashes. Remove redundant npm install step - only install the package directly. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Define qa-steps anchor once and reuse it for both qa-linux and qa-windows, following the same pattern as Cirrus CI configuration. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Define steps anchor in qa-linux and reference it in qa-windows, following GitHub Actions YAML anchor conventions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
SCANNPM-117
Conflicts:
.cirrus.yml
Part of