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

Fix links and update lychee in CI #51

Merged
merged 6 commits into from
Mar 4, 2024
Merged

Conversation

G1gg1L3s
Copy link
Contributor

@G1gg1L3s G1gg1L3s commented Mar 1, 2024

This PR fixes all broken links from #50. Also, I took a chance to update lychee version and fix broken links after it.

G1gg1L3s added 6 commits March 1, 2024 20:41
The link was already on web.archive. However, we need to pin to
exact date because the latest snapshot is for 404 page.
Here we have eprint link so let's use it. It's fast, free and has
HTTPS support :)
Unfortunately, the link is not working anymore and there are no free
resources with this work. So, let's use web.archive
Maybe it will help CI because as for now it fails with this link.
The previous link was used unsafe TLS features, so lychee was not
happy about it.
@G1gg1L3s
Copy link
Contributor Author

G1gg1L3s commented Mar 1, 2024

Seems like NIST websites are under maintenance, so let's wait some time and rerun CI.

@pFarb
Copy link
Owner

pFarb commented Mar 1, 2024

You are my saviour @G1gg1L3s! Thank you for maintaining the links. I agree let's wait for NIST to recover and try again.

@G1gg1L3s
Copy link
Contributor Author

G1gg1L3s commented Mar 4, 2024

@pFarb, could you please rerun the CI? I just run it on my fork and it seems to be ok - https://github.com/G1gg1L3s/awesome-crypto-papers/actions/runs/8116216183

@pFarb
Copy link
Owner

pFarb commented Mar 4, 2024

Let's wait

@pFarb
Copy link
Owner

pFarb commented Mar 4, 2024

@G1gg1L3s It's working, I'm merging

@pFarb pFarb merged commit 4541603 into pFarb:master Mar 4, 2024
1 check passed
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