Skip to content

Conversation

ryanthemanuel
Copy link
Collaborator

@ryanthemanuel ryanthemanuel commented Sep 11, 2025

Additional details

Handle a few scenarios with the experiment better.

image image

Steps to test

How has the user experience changed?

PR Tasks

@ryanthemanuel ryanthemanuel changed the base branch from develop to feat/cy-prompt September 11, 2025 21:18
@ryanthemanuel ryanthemanuel self-assigned this Sep 11, 2025
@ryanthemanuel
Copy link
Collaborator Author

We can probably remove some dead code on the services side at some point with this change.

@ryanthemanuel ryanthemanuel changed the title Ryanm/fix/handle experimental config better fix: (cy.prompt) handle when the prompt is executed by in CT or when the experiment is not enabled Sep 11, 2025
cursor[bot]

This comment was marked as outdated.

Copy link

cypress bot commented Sep 11, 2025

cypress    Run #65530

Run Properties:  status check passed Passed #65530  •  git commit fc0fdb2115: fix snapshot
Project cypress
Branch Review ryanm/fix/handle-experimental-config-better
Run status status check passed Passed #65530
Run duration 20m 06s
Commit git commit fc0fdb2115: fix snapshot
Committer Ryan Manuel
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 9
Tests that did not run due to a developer annotating a test with .skip  Pending 1102
Tests that did not run due to a failure in a mocha hook  Skipped 4
Tests that passed  Passing 26686
View all changes introduced in this branch ↗︎

Warning

Partial Report: The results for the Application Quality reports may be incomplete.

UI Coverage  44.99%
  Untested elements 187  
  Tested elements 157  
Accessibility  97.9%
  Failed rules  4 critical   8 serious   2 moderate   2 minor
  Failed elements 101  

@ryanthemanuel ryanthemanuel merged commit 2df3598 into feat/cy-prompt Sep 16, 2025
85 of 89 checks passed
@ryanthemanuel ryanthemanuel deleted the ryanm/fix/handle-experimental-config-better branch September 16, 2025 04:46
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