Skip to content

Use html5 writer from docutils#3432

Open
nkr0 wants to merge 1 commit intogetpelican:mainfrom
nkr0:html5
Open

Use html5 writer from docutils#3432
nkr0 wants to merge 1 commit intogetpelican:mainfrom
nkr0:html5

Conversation

@nkr0
Copy link
Copy Markdown
Contributor

@nkr0 nkr0 commented Dec 2, 2024

Pull Request Checklist

Resolves: #3004

  • Ensured tests pass and (if applicable) updated functional test output
  • Conformed to code style guidelines by running appropriate linting tools
  • Added tests for changed code
  • Updated documentation for changed code

@justinmayer justinmayer added this to the Pelican 5.0 milestone Dec 2, 2024
@nkr0 nkr0 force-pushed the html5 branch 2 times, most recently from 7021848 to e71a655 Compare January 30, 2025 22:21
Copy link
Copy Markdown
Member

@pauloxnet pauloxnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@justinmayer
Copy link
Copy Markdown
Member

As I mentioned in the linked issue, I suspect this change might be best introduced in the context of a major version, which is why I have targeted it for an upcoming 5.0 release. I hope to organize a sprint sometime soon so we can make that happen 😅

@nkr0
Copy link
Copy Markdown
Contributor Author

nkr0 commented Jan 31, 2025

As I mentioned in the linked issue, I suspect this change might be best introduced in the context of a major version, which is why I have targeted it for an upcoming 5.0 release. I hope to organize a sprint sometime soon so we can make that happen 😅

Yes, I remember. I did a rebase yesterday to fix a build failure after merging main

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.

Switch to Docutils HTML5 writer instead of the HTML4 one

3 participants