Skip to content

Add gitignore template linter - #4943

Merged
dooleydevin merged 4 commits into
mainfrom
dooleydevin/add-gitignore-template-linter
Nov 14, 2026
Merged

dooleydevin merged 4 commits into
mainfrom
dooleydevin/add-gitignore-template-linter

Conversation

@dooleydevin

@dooleydevin dooleydevin commented Nov 14, 2026 •

Copy link
Copy Markdown
Collaborator

Uses pre-commit-hooks tooling to handle linting of trailing whitespace and consistent newlines at the end of files. Uses dos2unix to normalize CRLF line endings while preserving literal carriage returns in gitignore patterns.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@dooleydevin
dooleydevin requested a review from a team as a code owner November 14, 2026 00:29
Copilot AI dynamic review requested due to automatic review settings November 14, 2026 00:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new required formatting workflow cannot pass because existing repository files have mixed line endings, trailing whitespace, and a missing final newline.

1 open issue
What changed in this PR

Adds a GitHub Actions workflow to lint all tracked .gitignore files for consistent line endings, trailing whitespace, and final newlines.

Changes:

  • Runs formatting checks on pull requests and manual dispatches.
  • Pins the GitHub Actions and pre-commit-hooks versions.
  • Recursively discovers tracked .gitignore files and reports formatter-generated diffs when checks fail.
File Description
.github/​workflows/​gitignore-formatting.yml Adds a GitHub Actions workflow that runs pinned formatting checks on all tracked .gitignore files and reports failures with the resulting diff.

Review effort: Lite (dynamic)

dooleydevin and others added 3 commits November 13, 2026 16:36
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@dooleydevin
dooleydevin merged commit b71db92 into main Nov 14, 2026
5 checks passed
@dooleydevin
dooleydevin deleted the dooleydevin/add-gitignore-template-linter branch November 14, 2026 00:54
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