-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[CI:DOCS] Fix docs/version-check always requesting updates #17620
[CI:DOCS] Fix docs/version-check always requesting updates #17620
Conversation
@edsantiago PTAL when you have time, this is not critical but it is causing the daily cirrus-cron job on |
e9bc79e
to
7d2c60d
Compare
Force-push: Rebased on #17621 + added revert. |
7d2c60d
to
79f391b
Compare
force-push: Fixed missing DCO |
/hold waiting-for-ed |
I'm going to defer this. It's impossible to review today, because there's a release in progress: tag without build.
And I'm totally fine with just removing this step. I don't know if it really adds any value. |
In fact, now that I've thought about it, this is likely the case. Not too long ago I updated the |
Correction: These markdown pages are about the released Windows installer, not the CI artifact. Still, I think Ed's right, including the version number and a direct link adds little value and a whole lot of (script) complications. Let's just remove it and the generic "release page" link is very likely good enough. |
79f391b
to
991c7e6
Compare
Force-push: Remove the script and update docs pages accordingly. |
As suggested by @edsantiago, the complex script and direct-link in the docs are adding little value while increasing the CI maintenance burden. Simply retire the script and strip the direct (versioned) links from the docs. Signed-off-by: Chris Evich <[email protected]>
f8543c0
to
0d21948
Compare
Force-push: Rebased and squashed. |
LGTM |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cevich, rhatdan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold cancel |
DEPENDS ON: #17621
As suggested by @edsantiago, the complex script and direct-link in the
docs are adding little value while increasing the CI maintenance burden.
Simply retire the script and strip the direct (versioned) links from the
docs.
Fixes (in cirrus-cron CI), this message always appearing & job failing:
Does this PR introduce a user-facing change?