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

Deleted unused code.md file #193

Merged
merged 2 commits into from
Sep 15, 2023
Merged

Deleted unused code.md file #193

merged 2 commits into from
Sep 15, 2023

Conversation

mariospr
Copy link
Member

@mariospr mariospr commented Sep 15, 2023

It has been unreachable under wpewebkit.org/code for a while, but now it's integrated in get-wpe.html, so let's remove it.

Site preview: https://igalia.github.io/wpewebkit.org/remove-code-page


Site preview: https://igalia.github.io/wpewebkit.org/remove-code-page/

It has been unreachable under wpewebkit.org/code for a while,
but now it's integrated in get-wpe.html, so let's remove it.
@meyerweb
Copy link
Member

Do we have a redirect from the old URL to the new one in place? Or anything showing the old URL is never visited? (I dislike just 404ing URLs.)

@mariospr mariospr enabled auto-merge (rebase) September 15, 2023 14:30
@mariospr
Copy link
Member Author

Do we have a redirect from the old URL to the new one in place? Or anything showing the old URL is never visited? (I dislike just 404ing URLs.)

A crawler I run in the past showed that code/ was only linked wrongly from explore-wpe.html in a "contact us" link that should be a mailto: link instead, but that has been fixed and with that there is no any other place where this is referenced (which I confirmed by grepping the code).

From what I can tell, that code/ URL was plainly unaccessible

@mariospr mariospr merged commit 7ee5819 into master Sep 15, 2023
1 check passed
@mariospr mariospr deleted the remove-code-page branch September 15, 2023 15:04
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