Skip to content

Disable incremental mode for cspell-action#81

Merged
Malcolmnixon merged 2 commits into
mainfrom
copilot/disable-incremental-files-only
Feb 10, 2026
Merged

Disable incremental mode for cspell-action#81
Malcolmnixon merged 2 commits into
mainfrom
copilot/disable-incremental-files-only

Conversation

Copilot AI commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

Pull Request

Description

Configured cspell-action to check all files instead of only changed files by setting incremental_files_only: false.

Change:

- name: Run spell checker
  uses: streetsidesoftware/cspell-action@v8
  with:
    files: '**/*.{md,cs}'
    incremental_files_only: false  # Added

This ensures comprehensive spell checking across the entire codebase on every workflow run, rather than limiting checks to modified files.

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

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

Linting

  • Markdown linter passes: npx markdownlint-cli2 "**/*.md" "#node_modules"
  • Spell checker passes: npx cspell "**/*.{md,cs}" --no-progress
  • YAML linter passes: yamllint .

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

Workflow configuration change only - no code or documentation changes required.

Original prompt

Please modify the streetsidesoftware/cspell-action invocation to disable incremental_files_only


✨ 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] Update cspell-action to disable incremental files only Disable incremental mode for cspell-action Feb 10, 2026
Copilot AI requested a review from Malcolmnixon February 10, 2026 12:11
@Malcolmnixon Malcolmnixon marked this pull request as ready for review February 10, 2026 12:17
@Malcolmnixon Malcolmnixon merged commit 0c5fd91 into main Feb 10, 2026
11 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/disable-incremental-files-only branch February 10, 2026 12:26
Malcolmnixon added a commit that referenced this pull request Mar 31, 2026
…eDotNetTool PR #81) (#111)

* Initial plan

* Reorganize SonarMark source into subsystem subfolders

- Move source files into Cli/, SelfTest/, SonarIntegration/, ReportGeneration/
- Update namespaces and using statements in all moved files
- Mirror the same layout in test project
- Move design docs and reqstream files into subsystem subdirectories
- Add subsystem overview design docs and requirements files
- Update docs/design/definition.yaml, requirements.yaml, .reviewmark.yaml

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

* Address review feedback: add using directive in Context.cs and use glob in reviewmark AllRequirements

Agent-Logs-Url: https://github.com/demaconsulting/SonarMark/sessions/a74327f2-5782-4fb9-8a8f-b304849242d7

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

* Add children links to requirements hierarchy (system→subsystem→unit)

Agent-Logs-Url: https://github.com/demaconsulting/SonarMark/sessions/83dd5df8-ecd6-4a58-bf4c-235a19a3fe5b

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

* Fix linting issues, correct test names, and add SonarQubeClient auth header tests

Agent-Logs-Url: https://github.com/demaconsulting/SonarMark/sessions/aa5c4ed5-8d7f-4d0b-a194-8b3d3982ba50

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
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