Skip to content

chore: delete cypress - #37310

Merged
pauldambra merged 4 commits into
masterfrom
chore/delete-cypress
Aug 29, 2025
Merged

chore: delete cypress#37310
pauldambra merged 4 commits into
masterfrom
chore/delete-cypress

Conversation

@pauldambra

Copy link
Copy Markdown
Member

i can't see anybody carrying on migrating these
and we don't run them

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

86 files reviewed, no comments

Edit Code Review Bot Settings | Greptile

@github-actions

github-actions Bot commented Aug 28, 2025

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 2.67 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 2.67 MB

compressed-size-action

@pauldambra
pauldambra requested a review from mariusandra August 28, 2025 18:08

@mariusandra mariusandra left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😢 🥲

@pauldambra
pauldambra enabled auto-merge (squash) August 29, 2025 08:26
@pauldambra
pauldambra merged commit a486343 into master Aug 29, 2025
177 checks passed
@pauldambra
pauldambra deleted the chore/delete-cypress branch August 29, 2025 08:28
@daibhin

daibhin commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

I'm seeing reliable failures of the 'System Status loaded' test (Example: https://github.com/PostHog/posthog/actions/runs/17328665970/job/49199191622?pr=37302)

Using your Playwright inspection trick @pauldambra I've tracked it down to a 500 response from the http://localhost:8000/api/instance_status/ API. Given this is mocked in the handlers.ts I'm wondering if it's somehow failing since removing Cypress

rafaeelaudibert added a commit that referenced this pull request Sep 2, 2025
Cypress was removed in #37310 but there are some stuff missing - the biggest offender is the broken pnpm-lock.yaml. we've removed 600 lines of it! That's 2% :)
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.

3 participants