Skip to content

Expand verify-docs: check all src/ files for CRLF and missing final newline; fix violations#205

Merged
eisenwave merged 2 commits into
mainfrom
copilot/expand-verify-docs-script
May 13, 2026
Merged

Expand verify-docs: check all src/ files for CRLF and missing final newline; fix violations#205
eisenwave merged 2 commits into
mainfrom
copilot/expand-verify-docs-script

Conversation

Copilot AI commented May 13, 2026

Copy link
Copy Markdown
Contributor

verify-docs.sh only enforced whitespace hygiene on .cow files. This extends coverage to all src/ files for two additional checks, and fixes the existing violations found.

Script changes (scripts/verify-docs.sh)

New loop over find src/ -type f before the existing .cow checks:

while IFS= read -r src; do
  if grep -Pq $'\r' "$src" 2>/dev/null; then
    echo "CRLF: $src contains CRLF line terminators"
    failed=1
  fi
  if [ -s "$src" ]; then
    last=$(tail -c 1 "$src" | od -An -tx1 | tr -d ' \n')
    if [ "$last" != "0a" ]; then
      echo "NO_FINAL_NL: $src does not end with a newline"
      failed=1
    fi
  fi
done < <(find src/ -type f)

File fixes

  • CRLF → LF: src/icon/{css,go,javascript,julia,kotlin,python,rust,typescript}.svg
  • Added final newline: src/icon/{javascript,css,typescript,swift}.svg, src/array-layout.bs, src/bitint-lewg-slides.cow, src/modular-arithmetic.bs, src/constexpr-to-string.bs

Copilot AI and others added 2 commits May 13, 2026 12:12
… newline; fix non-compliant files

Agent-Logs-Url: https://github.com/eisenwave/cpp-proposals/sessions/da2373aa-05a1-47a2-83d9-575b4cc50bbe

Co-authored-by: eisenwave <22040976+eisenwave@users.noreply.github.com>
@eisenwave eisenwave marked this pull request as ready for review May 13, 2026 12:16
@eisenwave eisenwave merged commit 5dafd7a into main May 13, 2026
2 checks passed
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