Skip to content

docs: replace Map of Zones report - #1610

Merged
evanorti merged 1 commit into
mainfrom
evanorti/map-of-zones-pdf-preview
Oct 26, 2026
Merged

evanorti merged 1 commit into
mainfrom
evanorti/map-of-zones-pdf-preview

Conversation

@evanorti

Copy link
Copy Markdown
Collaborator

Description

The PDF added in #1608 needed to be changed.

It is also exempted from the repo-wide LFS rule for PDFs. GitHub shows the LFS banner and a download button for LFS-backed files and never an inline preview, and this report is meant to be read in the blob view.


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
  • Include changelog entry when appropriate (e.g. chores should be omitted from changelog).
  • Wrote unit and e2e tests if relevant.
  • [ x] Updated documentation (docs/) if anything is changed.
  • Added godoc comments if relevant.
  • Self-reviewed Files changed in the GitHub PR explorer.
  • [ x] Provide a conventional commit message to follow the repository standards.

The PDF added in #1608 had been through a UTF-8 round-trip before it was
committed: every non-UTF-8 byte had become U+FFFD, which broke the xref
table and left all seven pages blank. This commits the intact export.

It is also exempted from the repo-wide LFS rule for PDFs. GitHub shows
the LFS banner and a download button for LFS-backed files and never an
inline preview, and this report is meant to be read in the blob view.

Signed-off-by: evanorti <87997759+evanorti@users.noreply.github.com>
@evanorti
evanorti requested a review from a team as a code owner October 26, 2026 16:41
@linear-code

linear-code Bot commented Oct 26, 2026

Copy link
Copy Markdown

FOU-2050

@greptile-apps

greptile-apps Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issues were found.

Summary

Replaces docs/MapOfZones_IBC_Report.pdf and exempts it from Git LFS so GitHub can show it inline.

  • The Map of Zones report opens as a GitHub preview.

Reviews (1) · Last reviewed commit: "docs: replace corrupted Map of Zones rep..." · Reviewed by Greptile

@evanorti
evanorti added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit eb30107 Oct 26, 2026
21 checks passed
@evanorti
evanorti deleted the evanorti/map-of-zones-pdf-preview branch October 26, 2026 17:06
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