From: Günter M. <mi...@us...> - 2022-01-20 09:57:10
|
Thank you for the patch. Most of it is applied now. Changing the IDN breaks the validity of generated XML documents: #test/functional/expected > xmllint standalone_rst_docutils_xml.xml --valid --noout standalone_rst_docutils_xml.xml:4: validity error : Validation failed: no DTD found ! Maybe this is also why flake8 failed? https://github.com/AA-Turner/docutils/runs/4877613719?check_suite_focus=true Commit 6/6 did not apply cleanly here and is also flagged "Some checks were not successful", so I leave it for now. --- ** [patches:#191] Update URLs ** **Status:** open **Group:** None **Created:** Thu Jan 20, 2022 04:40 AM UTC by Adam Turner **Last Updated:** Thu Jan 20, 2022 04:40 AM UTC **Owner:** nobody Documentation / minor change. There are a lot of old URLs in the repo -- e.g. `docutils.sf.net`, `docutils.sourceforge.net` -- whilst these currently still work, we should update them all to the canonical `https://docutils.sourceforge.io` This patch does that, and also generally updates several other links to `https://` -- I tested each domain, so didn't update some old sourceforge domains that don't support `https://`. I also updated the `docutils.dtd` IDN in a single commit, so it can be dropped if wanted. I'd certainly reccomend applying the first 4 commits, and ideally the entire patch. https://github.com/AA-Turner/docutils/pull/10 // https://github.com/AA-Turner/docutils/pull/10.patch A --- Sent from sourceforge.net because doc...@li... is subscribed to https://sourceforge.net/p/docutils/patches/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/docutils/admin/patches/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |