Skip to content

Infra: Improve email and link processing and rendering in headers #2467

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hugovk merged 7 commits into from
Apr 19, 2022
Merged

Infra: Improve email and link processing and rendering in headers #2467

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5ee6938
Infra: Include mailing list link type in text for clarity, & refactor
CAM-Gerlach Mar 24, 2022
f9d14d3
Infra: Support mailing list info pages as well as archives
CAM-Gerlach Mar 24, 2022
77debe1
Infra: Pretty-format Discourse categories/threads in headers as well
CAM-Gerlach Mar 25, 2022
a2d50c4
Infra: Show friendly-name of target in Post-History link hover text
CAM-Gerlach Mar 25, 2022
e10e775
Infra: Automatically link list addresses to list pages in Discussions-To
CAM-Gerlach Mar 25, 2022
ebed4ce
Infra: Use simpler check for title in Discourse URL header processing
CAM-Gerlach Mar 27, 2022
a0bc50f
Infra: Automatically trim trailing commas & whitespace in header values
CAM-Gerlach Apr 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Infra: Support mailing list info pages as well as archives
  • Loading branch information
@CAM-Gerlach
CAM-Gerlach committed Apr 17, 2022
commit f9d14d3cbcc1150dcde010fb193a340e9e15c357
11 changes: 11 additions & 0 deletions pep_sphinx_extensions/pep_processor/transforms/pep_headers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,12 +110,23 @@ def _process_list_url(list_url: str) -> tuple[str, str]:
if len(parts) > 6 and parts[6] in {"message", "thread"}:
url_type = parts[6]

# Mailman3 list info structure is
# https://mail.python.org/mailman3/lists/<list_name>.python.org/
elif "mailman3" in parts:
list_name = (
parts[parts.index("mailman3") + 2].removesuffix(".python.org"))

# Pipermail (Mailman) archive structure is
# https://mail.python.org/pipermail/<list_name>/<month>-<year>/<id>
elif "pipermail" in parts:
list_name = parts[parts.index("pipermail") + 1]
url_type = "message" if len(parts) > 6 else "list"

# Mailman listinfo structure is
# https://mail.python.org/mailman/listinfo/<list_name>
elif "listinfo" in parts:
list_name = parts[parts.index("listinfo") + 1]

# Not a link to a mailing list, message or thread
else:
raise ValueError("Not a link to a mailing list, message or thread")
Expand Down