[PR #1379] [MERGED] chore(format): apply bibtex-tidy + EOF-fixer residuals (unblock Book badge) #6504

Closed
opened 2026-04-21 22:22:50 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1379
Author: @profvjreddi
Created: 4/17/2026
Status: Merged
Merged: 4/17/2026
Merged by: @profvjreddi

Base: devHead: fix/ci-autofix-residuals


📝 Commits (1)

  • 4aa81ea chore(format): apply bibtex-tidy + end-of-file-fixer residuals

📊 Changes

5 files changed (+802 additions, -640 deletions)

View changed files

📝 interviews/paper/references.bib (+481 -365)
📝 interviews/vault/REVIEWS.md (+0 -2)
📝 periodic-table/paper/references.bib (+320 -272)
📝 site/newsletter/drafts/essay-03-builders-gap.md (+0 -1)
📝 tinytorch/site/references.bib (+1 -0)

📄 Description

Summary

After #1375 merged, re-ran book-validate-dev.yml manually. Pre-commit still fails on two auto-fix hooks because recent merges introduced content that was never committed in canonical form:

  • Global: Fix end of file newlines — 2 files (interviews/vault/REVIEWS.md, site/newsletter/drafts/essay-03-builders-gap.md)
  • Repo: Tidy bibtex files (format) — 3 files (interviews/paper/references.bib, periodic-table/paper/references.bib, tinytorch/site/references.bib)

Running both hooks locally and committing the output. Purely mechanical — no semantic changes.

Verification

  • Both hooks idempotent on second run (exit 0)
  • book/tools/bib_lint.py --check --all → 0 NEW errors (67 grandfathered)
  • Large line-count delta in the two bib files is from the --align=space --sort=key --sort-fields --wrap=100 config, not content change

Test plan

  • CI pre-commit goes green on this branch
  • After merge, book-validate-dev.yml passes pre-commit → container matrix builds run → Book badge recovers

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/1379 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 4/17/2026 **Status:** ✅ Merged **Merged:** 4/17/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `fix/ci-autofix-residuals` --- ### 📝 Commits (1) - [`4aa81ea`](https://github.com/harvard-edge/cs249r_book/commit/4aa81eaea3e359372c44b6b203a98214d8622784) chore(format): apply bibtex-tidy + end-of-file-fixer residuals ### 📊 Changes **5 files changed** (+802 additions, -640 deletions) <details> <summary>View changed files</summary> 📝 `interviews/paper/references.bib` (+481 -365) 📝 `interviews/vault/REVIEWS.md` (+0 -2) 📝 `periodic-table/paper/references.bib` (+320 -272) 📝 `site/newsletter/drafts/essay-03-builders-gap.md` (+0 -1) 📝 `tinytorch/site/references.bib` (+1 -0) </details> ### 📄 Description ## Summary After #1375 merged, re-ran `book-validate-dev.yml` manually. Pre-commit still fails on two auto-fix hooks because recent merges introduced content that was never committed in canonical form: - `Global: Fix end of file newlines` — 2 files (interviews/vault/REVIEWS.md, site/newsletter/drafts/essay-03-builders-gap.md) - `Repo: Tidy bibtex files (format)` — 3 files (interviews/paper/references.bib, periodic-table/paper/references.bib, tinytorch/site/references.bib) Running both hooks locally and committing the output. Purely mechanical — no semantic changes. ## Verification - Both hooks idempotent on second run (exit 0) - `book/tools/bib_lint.py --check --all` → 0 NEW errors (67 grandfathered) - Large line-count delta in the two bib files is from the `--align=space --sort=key --sort-fields --wrap=100` config, not content change ## Test plan - [ ] CI pre-commit goes green on this branch - [ ] After merge, `book-validate-dev.yml` passes pre-commit → container matrix builds run → Book badge recovers --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-21 22:22:50 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#6504