Skip to content

Conversation

@2bndy5
Copy link
Contributor

@2bndy5 2bndy5 commented Sep 6, 2025

  • include spell check config in this repo
  • fixed a misspelling
  • introduce c-spell pre-commit hook
  • include conventional-commit config
  • adjust markdown lint ignore in profile/README

- include spell check config in this repo
- fixed a misspelling
- introduce c-spell pre-commit hook
- include conventional-commit config
@2bndy5 2bndy5 marked this pull request as ready for review September 6, 2025 00:39
@2bndy5 2bndy5 requested a review from shenxianpeng September 6, 2025 00:39
Careful review should be paid when using
`pre-commit autoupdate` (which only uses tags).
2bndy5 added a commit to cpp-linter/cpp-linter that referenced this pull request Sep 6, 2025
See cpp-linter/.github#50 for new release workflow

See cpp-linter/.github#49 for changes to pre-commit workflow

I generated the CHANGELOG doc which summarizes all previous releases.
2bndy5 added a commit to cpp-linter/cpp-linter that referenced this pull request Sep 6, 2025
See cpp-linter/.github#50 for new release workflow

See cpp-linter/.github#49 for changes to pre-commit workflow

I generated the CHANGELOG doc which summarizes all previous releases.
@2bndy5
Copy link
Contributor Author

2bndy5 commented Sep 6, 2025

I need these changes in main to move other PRs forward

have to use org-repo main branch for committed config
considering if there's no other need for node.js installed
@2bndy5 2bndy5 merged commit 724d561 into main Sep 7, 2025
4 checks passed
@2bndy5 2bndy5 deleted the check-pr-title branch September 7, 2025 13:55
2bndy5 added a commit to cpp-linter/cpp-linter that referenced this pull request Sep 7, 2025
See cpp-linter/.github#50 for new release workflow

See cpp-linter/.github#49 for changes to pre-commit workflow

I generated the CHANGELOG doc which summarizes all previous releases.
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.

3 participants