Skip to content

Repo: Remove end-to-end website tests (for now) #7729

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

Closed
JoshuaKGoldberg opened this issue Oct 8, 2023 · 0 comments · Fixed by #7730
Closed

Repo: Remove end-to-end website tests (for now) #7729

JoshuaKGoldberg opened this issue Oct 8, 2023 · 0 comments · Fixed by #7730
Assignees
Labels
accepting prs Go ahead, send a pull request that resolves this issue repo maintenance things to do with maintenance of the repo, and not with code/docs

Comments

@JoshuaKGoldberg
Copy link
Member

Suggestion

Our end-to-end tests for the website are almost always ❌ broken in some way. Most of the time it's at least that their run+deploy machinery is broken on forks. Oftentimes they also become flaky for everyone. It's annoying, inconvenient, and very confusing+offputting for contributors. Especially new folks!

We'd added those tests back when the website was under much more active development. But the website doesn't change as much anymore. I don't think the tests are still worth the pain of maintenance.

A few relevant links: #4362, #5814, #5579.

Inspired by @trentmwillis's talk at Revo.js 2023 🙂

@JoshuaKGoldberg JoshuaKGoldberg added triage Waiting for team members to take a look repo maintenance things to do with maintenance of the repo, and not with code/docs accepting prs Go ahead, send a pull request that resolves this issue and removed triage Waiting for team members to take a look labels Oct 8, 2023
@JoshuaKGoldberg JoshuaKGoldberg self-assigned this Oct 8, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepting prs Go ahead, send a pull request that resolves this issue repo maintenance things to do with maintenance of the repo, and not with code/docs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant