Skip to content

Docs: remove expired links from old jquery source #4997

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 7, 2022
Merged

Docs: remove expired links from old jquery source #4997

merged 1 commit into from
Jan 7, 2022

Conversation

timmywil
Copy link
Member

@timmywil timmywil commented Jan 6, 2022

Ref gh-4981
Ref gh-4991

Summary

We had removed the web archive links, but these links are still in the repo. Technically, the test folder is not distributed to releases, but we may as well remove them anyway.

Checklist

Sorry, something went wrong.

mgol
mgol approved these changes Jan 7, 2022
View reviewed changes
Copy link
Member

@mgol mgol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be better to update to a version that doesn't contain these links... but there's no such version available yet. Maybe we'll do the update after some time. LGTM

@timmywil
Copy link
Member Author

timmywil commented Jan 7, 2022

Thanks, I'll make a companion PR for 3.x-stable.

@timmywil timmywil mentioned this pull request Jan 7, 2022
Merged
3 tasks
@timmywil timmywil merged commit ed066ac into jquery:main Jan 7, 2022
@timmywil timmywil deleted the link-removal branch January 7, 2022 17:44
@mgol mgol added this to the 4.0.0 milestone Jan 7, 2022
@mgol mgol added the Docs label Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgol mgol

Assignees
No one assigned
Labels
Docs
Milestone
4.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@timmywil @mgol