-
Notifications
You must be signed in to change notification settings - Fork 933
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
Fix metadata-up-to-date lint script #12222
Conversation
Signed-off-by: Simon Deziel <[email protected]>
…eturns non-0 Signed-off-by: Simon Deziel <[email protected]>
@ru-fu it seems this link is sometimes returning a 403:
Which is easy enough to trigger:
Is there an exclusion mechanism we can use to ignore it? |
I've never seen that before - maybe it was just flaky yesterday when you tested? |
12b562b
to
49a0d24
Compare
I can't get the 403 today so it probably was a one off. As such, I dropped all the link checker changes from the PR. Thanks @ru-fu! |
Hmm, this time it's not a 403 but a TLS handshake failure:
And it's not just us apparently: https://www.ssllabs.com/ssltest/analyze.html?d=docutils.sourceforge.io#12098 If the doc is available from another source, maybe we should switch. |
Hmm, this is the main source for it, I would be surprised if it was available somewhere else. |
The script would abort when a delta between "before" and "after" was noticed because
diff
would return with a non-0
code. Fix that so that we can see the delta if there is one and use "unified" mode.