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

Solve some polyfill link issues #2576

Merged
merged 6 commits into from
Jun 24, 2020
Merged

Solve some polyfill link issues #2576

merged 6 commits into from
Jun 24, 2020

Conversation

Markel
Copy link
Contributor

@Markel Markel commented Jun 22, 2020

As many of the links go back years and years some websites are down or they updated their URLs, the PR changes the URLs or deletes the polyfills to accommodate the changes. Also it updates some links from http to https. It is important to notice that most github links haven't been checked because my link check extension was always hitting the GitHub API rate limit 🙃.

@codecov
Copy link

codecov bot commented Jun 22, 2020

Codecov Report

Merging #2576 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2576   +/-   ##
=======================================
  Coverage   95.15%   95.15%           
=======================================
  Files           5        5           
  Lines         165      165           
=======================================
  Hits          157      157           
  Misses          8        8           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 771354e...0286ce0. Read the comment docs.

lib/polyfills.json Outdated Show resolved Hide resolved
@Markel Markel requested a review from rejas June 24, 2020 19:16
@rejas rejas merged commit d31f4a0 into Modernizr:master Jun 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants