Skip to content

Normalize workflow YAML line endings to CRLF per .editorconfig - #446

Merged
ptr727 merged 1 commit into
developfrom
normalize-workflow-eol
Jun 24, 2026
Merged

Normalize workflow YAML line endings to CRLF per .editorconfig#446
ptr727 merged 1 commit into
developfrom
normalize-workflow-eol

Conversation

@ptr727

@ptr727 ptr727 commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Six workflow/config files had drifted to LF, but .editorconfig mandates CRLF for *.{yml,yaml}. This normalizes them to CRLF so the editor/lint config and the files agree.

Files:

  • .github/dependabot.yml
  • .github/workflows/build-docker-task.yml
  • .github/workflows/merge-bot-pull-request.yml
  • .github/workflows/publish-release.yml
  • .github/workflows/run-codegen-pull-request-task.yml
  • .github/workflows/test-pull-request.yml

The LF→CRLF flip rewrites every line, which is why this is isolated in its own change (per the template's line-ending governance). No content changes; YAML still parses.

Other tracked files either already satisfy their .editorconfig EOL rule or have no rule (Dockerfiles and s6-overlay init scripts are intentionally left as-is — the Dockerfiles are generated, and the init scripts must stay LF to run in-container).

These six workflow/config files had drifted to LF; .editorconfig mandates
CRLF for *.{yml,yaml}. Isolated in its own commit (per the template's
line-ending governance) since the LF->CRLF flip rewrites every line.
Copilot AI review requested due to automatic review settings June 24, 2026 02:09

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 wasn't able to review any files in this pull request.

@ptr727
ptr727 merged commit e20779e into develop Jun 24, 2026
9 checks passed
@ptr727
ptr727 deleted the normalize-workflow-eol branch June 24, 2026 04:38
ptr727 added a commit that referenced this pull request Jun 24, 2026
Maintenance promotion of `develop` to `main`. `version.json` is
unchanged (2.13 on both branches), so NBGV advances only `main`'s height
— no minor bump.

## What's promoted

A single change since the last promotion (#445):
- **Workflow YAML line-ending normalization** (#446):
`.github/dependabot.yml` and the five `.github/workflows/*.yml` that had
drifted to LF are normalized to CRLF, matching `.editorconfig`'s
`*.{yml,yaml}` rule. Whitespace-only; no content or behavior change.

## Notes

- Direct develop->main merge is clean (no conflicts; `main` did not
touch these files since the merge-base).
- Merge method is **merge-commit**, per branch protection.
- Merging does **not** publish — publishing is the weekly schedule /
manual dispatch only.
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