Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Generate a release schedule as an.ics file#4705

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.

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

Already on GitHub?Sign in to your account

Open
AA-Turner wants to merge11 commits intopython:main
base:main
Choose a base branch
Loading
fromAA-Turner:ics

Conversation

@AA-Turner
Copy link
Member

@AA-TurnerAA-Turner commentedNov 13, 2025
edited
Loading

This is derived from Hugo's#4704; see that PR for context.

This PR is an attempt to implement@sethmlarson'ssuggestion -- attempting to do this via review comments wasn't really going to work!

The ICS generation is now much simpler with no dependencies, simply outputting the required strings. I've also limited past events to 7 years to keep the file size smaller (for ~18months of pre-releases and a five-year lifespan). Realistically this could be much shorter, but I've been conservative.

A


📚 Documentation preview 📚:https://pep-previews--4705.org.readthedocs.build/release-schedule.ics

@AA-TurnerAA-Turner requested a review froma team as acode ownerNovember 13, 2025 20:03
@AA-TurnerAA-Turner added the infraCore infrastructure for building and rendering PEPs labelNov 13, 2025
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
AA-Turnerand others added5 commitsNovember 22, 2025 13:26
Comment on lines 74 to 79
# Disabled: no support for comments as at 2025-11-27.
# - repo: https://github.com/tox-dev/tox-ini-fmt
# rev: 1.4.1
# hooks:
# - id: tox-ini-fmt
# name: "Format tox.ini"
Copy link
MemberAuthor

Choose a reason for hiding this comment

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

cc@hugovk per#4715 (comment). The other option is to keeptox-ini-fmt but remove the comments entirely. What do you prefer?

Copy link
Member

Choose a reason for hiding this comment

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

Let's keep tox-ini-fmt and remove the comments, the comments are quite self-explanatory.

Copy link
Member

Choose a reason for hiding this comment

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

First time I encounter the need for autoformatting tox.ini !

Copy link
Member

Choose a reason for hiding this comment

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

It makes maintenance much easier when you can compare across repos and all the similar stuff is, well, similar. Like sections and keys in the same order.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@merwokmerwokmerwok left review comments

@hugovkhugovkhugovk left review comments

@encukouencukouAwaiting requested review from encukou

@sethmlarsonsethmlarsonAwaiting requested review from sethmlarson

@CAM-GerlachCAM-GerlachAwaiting requested review from CAM-Gerlach

Assignees

No one assigned

Labels

infraCore infrastructure for building and rendering PEPs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AA-Turner@merwok@hugovk

[8]ページ先頭

©2009-2025 Movatter.jp