Skip to content
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

Increase outdated build delay to two weeks #12063

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

Warriorrrr
Copy link
Member

Our old upstream loved to always randomly shift this value to arbitrary amounts of days, but since going stable this hasn't been touched yet. Two weeks seems like a good middle ground to keep it at (though I wonder if this message is even still needed).

The "as per instructions" part also seems like a holdout from spigot, since our downloads page has no instructions.

@Warriorrrr Warriorrrr requested a review from a team as a code owner February 3, 2025 21:33
@Doc94
Copy link
Contributor

Doc94 commented Feb 4, 2025

nice moment for in future improvement this? or just remove?
also can be good remove the Paper comments in that place?

@electronicboy
Copy link
Member

In the future we can maybe just switch this towards actually querying the API to print if an update is available post server init, but, that's low priority

@electronicboy electronicboy merged commit 742968e into PaperMC:main Feb 4, 2025
4 checks passed
@Warriorrrr Warriorrrr deleted the fix/outdated-build-delay branch February 4, 2025 18:37
Ifiht added a commit to Ifiht/Aincrad that referenced this pull request Feb 8, 2025
[ci skip] Increase outdated build delay to two weeks (PaperMC#12063)
Ifiht pushed a commit to Ifiht/Aincrad that referenced this pull request Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

4 participants