Skip to content

Updating Index links to migrated migration guides#558

Merged
mohawk2 merged 1 commit into
PDLPorters:masterfrom
duffee:POD-Index
Jun 26, 2026
Merged

Updating Index links to migrated migration guides#558
mohawk2 merged 1 commit into
PDLPorters:masterfrom
duffee:POD-Index

Conversation

@duffee

@duffee duffee commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Migration guides moved to PDL::Guide::Migration::*. Update Index.pod to match, with reordering to maintain alphabetic order.

@coveralls

coveralls commented Jun 24, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 70.486% (+1.0%) from 69.505% — duffee:POD-Index into PDLPorters:master

@mohawk2

mohawk2 commented Jun 24, 2026

Copy link
Copy Markdown
Member

Thank you for this. Please can you confirm this is the result of make doctest instead of manual editing? Otherwise it's at risk of being overwritten and/or wrong.

The whole doc-generation including pdldoc.db still needs bringing into the normal build process rather than being only done at doctest or install, which will be easier thanks to the recent changes that added t/doc.t. Volunteers to finish that up welcome! The pdldoc.db would be built with every make under lib/ (with a dep on all the *.pm files), and copied into blib.

changes made using `make doctest`
@duffee

duffee commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Sorry. First commit done by hand in ignorance and because of make issues, I had to build a container to run make doctest which I've just squashed and pushed.

in the meantime, I made a few corrections to Luis' 5yo Dockerfile which I'm getting the hang of.

@mohawk2

mohawk2 commented Jun 26, 2026

Copy link
Copy Markdown
Member

That's great, thank you! Let me know if you want help with updating doc generation as mentioned above.

@mohawk2
mohawk2 merged commit 823c6b3 into PDLPorters:master Jun 26, 2026
107 of 110 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.

3 participants