-
Notifications
You must be signed in to change notification settings - Fork 425
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
TEZ-4464: Update some links to https #260
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
thanks for this contribution @raboof , can you please create jira also? |
13ac301
to
9b20e8a
Compare
no tests were added because adding a link checker to the CI seems beyond the scope of this PR.
I manually followed most updated links to verify the linked sites indeed support https. I did not verify the pages still serve useful content.
Sure, done! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
markdownlint issues are not introduced by this patch, LGTM +1
No description provided.