Skip to content

Conversation

marktnoonan
Copy link
Contributor

Wrapping a couple of content changes together here in the App Quality area:

  • Release of data-a11y-ignore attribute
  • Publication of required CI environment variables for each supported provider for the Results API. This will help customers with getting started, debugging their code, and passing the right data through in more complicated setups that don't work out of the box.

Copy link

cypress bot commented Oct 15, 2025

cypress-documentation    Run #723

Run Properties:  status check passed Passed #723  •  git commit 294c00aa07: update AQ content
Project cypress-documentation
Branch Review marktnoonan/app-quality-updates-2
Run status status check passed Passed #723
Run duration 04m 32s
Commit git commit 294c00aa07: update AQ content
Committer marktnoonan
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 311
View all changes introduced in this branch ↗︎
UI Coverage  10.76%
  Untested elements 2523  
  Tested elements 3  
Accessibility  97.21%
  Failed rules  1 critical   3 serious   4 moderate   0 minor
  Failed elements 28  

@@ -0,0 +1,163 @@
The `@cypress/extract-cloud-results` helper cross-references some environment variables from where it is executed with ones that were present when a Cypress Cloud run was recorded. This allows for automatically detecting the correct Cloud run when the Results API is invoked from the same CI context as a given run (as is the case in the above examples).
Copy link
Member

Choose a reason for hiding this comment

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

This doc has a pretty big potential for becoming stale. It'd be great if we could somehow reference code to prevent this.

Copy link
Contributor Author

@marktnoonan marktnoonan Oct 16, 2025

Choose a reason for hiding this comment

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

Thanks for the review! Yeah I agree this is tricky. So far these haven't changed in the last year+. I was thinking we could link to this as the source of truth in the cloud internal readme for this package so that at least there is a cross reference if we ever do update these.

@marktnoonan marktnoonan merged commit 213c1f6 into main Oct 16, 2025
13 checks passed
@marktnoonan marktnoonan deleted the marktnoonan/app-quality-updates-2 branch October 16, 2025 17:08
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