Skip to content

Align build.yaml shell command formatting with SarifMark conventions#126

Merged
Malcolmnixon merged 2 commits intomainfrom
copilot/fix-build-yaml-commands
Jan 28, 2026
Merged

Align build.yaml shell command formatting with SarifMark conventions#126
Malcolmnixon merged 2 commits intomainfrom
copilot/fix-build-yaml-commands

Conversation

Copy link
Contributor

Copilot AI commented Jan 28, 2026

Pull Request

Description

Standardized shell command formatting in .github/workflows/build.yaml to match patterns used in the SarifMark repository.

Changes:

  • CodeQL job: Applied run: > multiline format to dotnet tool restore and dotnet restore
  • build-docs job: Applied run: > multiline format to dotnet tool restore and npm install
  • Display steps: Added explicit shell: bash directive to commands using echo and cat

All dotnet/npm commands now use consistent run: > formatting. Display commands use shell: bash with run: | for multiline echo/cat operations.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code quality improvement

Related Issues

N/A

Pre-Submission Checklist

Before submitting this pull request, ensure you have completed the following:

Build and Test

  • Code builds successfully: dotnet build --configuration Release
  • All tests pass: dotnet test --configuration Release
  • Code produces zero warnings

Code Quality

  • Code formatting is correct: dotnet format --verify-no-changes
  • New code has appropriate XML documentation comments
  • Static analyzer warnings have been addressed

Quality Checks

Please run the following checks before submitting:

  • Spell checker passes: cspell "**/*.{md,cs}"
  • Markdown linter passes: markdownlint "**/*.md"
  • YAML linter passes: yamllint '**/*.{yml,yaml}'

Testing

  • Added unit tests for new functionality
  • Updated existing tests if behavior changed
  • All tests follow the AAA (Arrange, Act, Assert) pattern
  • Test coverage is maintained or improved

Documentation

  • Updated README.md (if applicable)
  • Updated ARCHITECTURE.md (if applicable)
  • Added code examples for new features (if applicable)
  • Updated requirements.yaml (if applicable)

Additional Notes

Formatting-only changes. No functional modifications to workflow behavior.

Original prompt

Please examine https://github.com/demaconsulting/SarifMark and make the shell commands of the build.yaml file consistent.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Make shell commands in build.yaml consistent Align build.yaml shell command formatting with SarifMark conventions Jan 28, 2026
Copilot AI requested a review from Malcolmnixon January 28, 2026 13:41
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