Skip to content

Add check against date placeholders #1580

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

senekor
Copy link
Contributor

@senekor senekor commented Apr 16, 2025

The date of publication often has to be adjusted at the last minute. This can easily be forgotten. This commit introduces a new workflow: Blog authors keep a placeholder as the date (9999/12/99) until shortly before publication. Setting the date is aided with automation. Lastly, A check triggered by the merge queue event is used to block PRs from being merged where the placeholder hasn't been replaced yet.

The date of publication often has to be adjusted at the last minute.
This can easily be forgotten. This commit introduces a new workflow:
Blog authors keep a placeholder as the date (9999/12/99) until shortly
before publication. Setting the date is aided with automation. Lastly,
A check triggered by the merge queue event is used to block PRs from
being merged where the placeholder hasn't been replaced yet.
@senekor senekor closed this Apr 16, 2025
@senekor
Copy link
Contributor Author

senekor commented Apr 16, 2025

(I meant to open this against my own fork, I keep forgetting that GitHub suggests opening a PR against upstream by default.)

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.

1 participant