Release v0.44 - #31
Merged
Merged
Conversation
Ref #15 Ref #21 Ref #22 ## Reasoning Three user-visible changes since 0.43, all already reviewed and merged: - TrueType text is searchable and copyable (#15). This is the one Koha is waiting on for its PDF work. - "Didn't find pages" on filename reuse is fixed (#21, #22), an eleven-year-old RT report. - CI covers three platforms again (#9, #10). Adds MANIFEST.SKIP, which the distribution did not have. Without it `make manifest` sweeps in the previous release tarball and docs/reviews/*, neither of which belongs on CPAN -- the review artifacts are a record of how a change was reviewed in this repository and are noise to anyone installing the module. docs/ci-pin-maintenance.md is shipped deliberately: it documents why CI is pinned and how to advance it, which is useful to anyone packaging this. The tarball is not committed. No previous release tarball is tracked either, and *.tar.gz is already in .gitignore, so committing this one would be the inconsistency rather than omitting it. Deliberately not included: the %form/%intAct/%image staleness in #28, which is pre-existing and unsafe to fix from the invalidation point added in #26, and the Text::PDF stream-length bug in #23, which needs a fix in Text::PDF itself. ## Verification Built with `make dist` and verified from the extracted tarball rather than the working tree: `perl Makefile.PL && make && make test` passes 33 tests, and a PDF generated by the shipped module round-trips through pdftotext, confirming the ToUnicode work survives packaging. Tarball contents checked by hand -- no review artifacts, no old tarball, no CI configuration. Version reads 0.44 consistently in Reuse.pm, Makefile.PL, META.yml and META.json.
Coverage Report for CI Build 30412078859Warning No base build found for commit Coverage: 34.09%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
Owner
Author
|
CI: 14/14 green — run 30412078859. Full matrix across ubuntu/macOS/Windows, plus coverage and perlcritic. Tarball is built and verified: Checking back before the upload, as agreed — the PAUSE step is yours. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version bump, Changes entry, and MANIFEST curation for the 0.44 release.
Ref #15
Ref #21
Ref #22
What ships
Three user-visible changes since 0.43, all already reviewed and merged:
/ToUnicodeCMap is generated per font, covering only the glyphs actually used. This is what Koha's PDF work needs.Didn't find pageswhen a source filename is reused within one session. An eleven-year-old RT report.MANIFEST.SKIP is new
This distribution had none, so
make manifestswept in the previous release tarball and everydocs/reviews/*artifact. Neither belongs on CPAN — the review artifacts record how a change was reviewed in this repository and are noise to anyone installing the module.docs/ci-pin-maintenance.mdis shipped deliberately: it explains why upstream CI refs are pinned and how to advance them, which is useful to anyone packaging this.The tarball is not committed
No previous release tarball is tracked, and
*.tar.gzis already in.gitignore— committing this one would be the inconsistency, not omitting it.PDF-Reuse-0.44.tar.gz(74 KB) is built locally and ready for PAUSE.Verification
Verified from the extracted tarball, not the working tree — packaging bugs only show up that way:
Then an end-to-end check that the shipped module still does the thing this release is for:
Tarball contents inspected by hand — no review artifacts, no old tarball, no CI config. Version reads
0.44consistently acrossReuse.pm,Makefile.PL,META.ymlandMETA.json.Tagged
v0.44locally; the tag pushes with the merge.Deliberately not in this release
%form/%intAct/%imagestaleness. Pre-existing, and unsafe to fix from the invalidation point added in Fix stale %processed cache when a filename is reused (#21, #22) #26 because those entries are consumed mid-document./ocaching.Load-bearing: no. Version metadata, changelog and packaging only; no code changes — all the code in this release was reviewed and merged in #25, #26 and #30.