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

Regard HTTP 403 links as "alive", for now #266

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

EliahKagan
Copy link
Collaborator

This reconfigures markdown-link-check to treat both HTTP 200 and HTTP 403 status codes as meaning the link is not broken. This has a high risk of false detection. It is a temporary workaround for the problem of pages on the OpenAI website (linked to from the readme) returning HTTP 403 when accessed from GitHub Actions runners.

This reconfigures markdown-link-check to treat both HTTP 200 and
HTTP 403 status codes as meaning the link is not broken. This has a
high risk of false detection. It is a temporary workaround for the
problem of pages on the OpenAI website (linked to from the readme)
returning HTTP 403 when accessed from GitHub Actions runners.
@EliahKagan EliahKagan merged commit d7b07d4 into dmvassallo:main Dec 4, 2023
12 of 14 checks passed
@EliahKagan EliahKagan deleted the links branch December 4, 2023 04:14
EliahKagan added a commit to EliahKagan/EmbeddingScratchwork that referenced this pull request Dec 4, 2023
The URL for the mamba installation page changed, and the old URL
does not work as a redirect. This updates it.

This change is analogous to:
EliahKagan/palgoviz#190

Together with dmvassallo#266, this should make the "Check Markdown" workflow
all-green again.
EliahKagan added a commit that referenced this pull request Dec 4, 2023
The URL for the mamba installation page changed, and the old URL
does not work as a redirect. This updates it.

This change is analogous to:
EliahKagan/palgoviz#190

Together with #266, this should make the "Check Markdown" workflow
all-green again.
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.

1 participant