Skip to content

Move the Repository to the Fleet LF Line-Ending Policy - #128

Merged
ptr727 merged 7 commits into
developfrom
resync/lf-line-endings
Sep 26, 2026
Merged

ptr727 merged 7 commits into
developfrom
resync/lf-line-endings

Conversation

@ptr727

@ptr727 ptr727 commented Sep 26, 2026

Copy link
Copy Markdown
Owner

Moves the repository to the fleet LF line-ending policy, and re-vendors the markdownlint config. This is the second PR of the hub resync (audit run 2026-09-26T02:19:51Z, hub 45669468).

What changes

  • .editorconfig, .editorconfig-checker.json, and .gitattributes are carried byte-for-byte from the hub (main 45669468): LF for every text file, CRLF only for *.bat and *.cmd, and * text=auto eol=lf so git normalizes on commit and checkout. The interim [.github/skills/**] LF rule is gone, since LF is now the default.
  • Every other tracked text file is renormalized to LF. git diff --ignore-cr-at-eol against develop lists only the three files above and .markdownlint-cli2.jsonc.
  • .markdownlint-cli2.jsonc is re-vendored to the current canonical. The audit classified it stale, matching a past hub revision.

Verified

  • The hub repo gate's eol and eol-coverage checks pass. They failed before.
  • editorconfig-checker, markdownlint, and actionlint are clean.
  • No .bat, .cmd, or .ps1 files are tracked. Dockerfile and .sh files were already LF.

The branch carries a merge of develop after #127 squash-merged, resolved to this branch's side, since the two differed only in line endings outside the files listed above.

🤖 Generated with Claude Code

ptr727 and others added 7 commits September 25, 2026 19:25
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ings

# Conflicts:
#	.editorconfig
#	.github/copilot-instructions.md
#	.github/dependabot.yml
#	AGENTS.md
#	AUDIT.md
#	CLAUDE.md
#	CODESTYLE.md
#	GOVERNANCE.md
#	OPERATIONS.md
#	WORKFLOW.md
#	host-tools.json
Copilot AI lite review requested due to automatic review settings September 26, 2026 03:07
@coderabbitai

coderabbitai Bot commented Sep 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 27 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: cce2f4e0-4bd8-4a2c-9d72-9685c27971c7

📥 Commits

Reviewing files that changed from the base of the PR and between 451d20d and 3c7d5cb.

📒 Files selected for processing (28)
  • .dockerignore
  • .editorconfig
  • .editorconfig-checker.json
  • .gitattributes
  • .github/copilot-instructions.md
  • .github/dependabot.yml
  • .gitignore
  • .markdownlint-cli2.jsonc
  • .vscode/tasks.json
  • AGENTS.md
  • AUDIT.md
  • CLAUDE.md
  • CODESTYLE.md
  • Docker/README.md
  • GOVERNANCE.md
  • HISTORY.md
  • LICENSE
  • OPERATIONS.md
  • README.md
  • VSCode-Server-DotNetCore.code-workspace
  • WORKFLOW.md
  • cspell.json
  • host-tools.json
  • repo-config/README.md
  • repo-config/ruleset-develop.json
  • repo-config/ruleset-main.json
  • repo-config/settings.json
  • version.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The changes are consistent with the stated goal (LF normalization + config re-vendor) and do not introduce functional code-path risk.

Review effort: Lite
Findings: None

What changed in this PR

This pull request aligns the repository with the fleet LF line-ending policy by updating Git’s normalization rules and renormalizing tracked text files, while also re-vendoring the repository’s markdownlint-cli2 configuration to the current canonical form.

Changes:

  • Adopt LF as the repository-wide default via .gitattributes / .editorconfig, with CRLF reserved for *.bat and *.cmd.
  • Update EditorConfig Checker configuration to exclude common generated cache directories.
  • Re-vendor .markdownlint-cli2.jsonc with canonical defaults (including explicit globs and node_modules ignore).
File Description
VSCode-Server-DotNetCore.code-workspace Line-ending normalization only.
version.json Line-ending normalization only.
repo-config/​settings.json Line-ending normalization only.
repo-config/​ruleset-main.json Line-ending normalization only.
repo-config/​ruleset-develop.json Line-ending normalization only.
repo-config/​README.md Line-ending normalization only.
README.md Line-ending normalization only.
OPERATIONS.md Line-ending normalization only.
LICENSE Line-ending normalization only.
host-tools.json Line-ending normalization only.
HISTORY.md Line-ending normalization only.
Docker/​README.md Line-ending normalization only.
cspell.json Line-ending normalization only.
CODESTYLE.md Line-ending normalization only.
CLAUDE.md Line-ending normalization only.
AUDIT.md Line-ending normalization only.
AGENTS.md Line-ending normalization only.
.vscode/​tasks.json Line-ending normalization only.
.markdownlint-cli2.jsonc Re-vendored markdownlint-cli2 configuration (rules + explicit globs/ignores).
.gitignore Line-ending normalization only.
.github/​dependabot.yml Line-ending normalization only.
.github/​copilot-instructions.md Line-ending normalization only.
.gitattributes Switch to * text=auto eol=lf with CRLF exceptions for *.bat/*.cmd.
.editorconfig-checker.json Add Exclude patterns for common cache/venv directories.
.editorconfig Switch to LF default; retain CRLF exception for *.bat/*.cmd.
.dockerignore Line-ending normalization only.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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