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

a11y: keep a11y disabled on executing cypress Calc tests #7549

Merged
merged 1 commit into from
Oct 29, 2023

Conversation

mcecchetti
Copy link
Contributor

Signed-off-by: Marco Cecchetti [email protected]
Change-Id: Ic661358cd47600ec9c3a329bef361b8087483017

@mcecchetti mcecchetti force-pushed the private/marco/a11y/test_patch_for_CI branch 2 times, most recently from ac4f221 to 7c72082 Compare October 27, 2023 20:04
@mcecchetti mcecchetti changed the title test patch for CI: not to merge test patch for CI Oct 27, 2023
@mcecchetti mcecchetti force-pushed the private/marco/a11y/test_patch_for_CI branch 2 times, most recently from 0da97c8 to a9cee38 Compare October 27, 2023 22:15
This patch is a temporary workaround for fixing CI failures.
In fact on CI with a11y enabled cypress fails to load spreadsheets.

Signed-off-by: Marco Cecchetti <[email protected]>
Change-Id: Ic661358cd47600ec9c3a329bef361b8087483017
@mcecchetti mcecchetti force-pushed the private/marco/a11y/test_patch_for_CI branch from a9cee38 to 68c4e52 Compare October 27, 2023 22:41
@mcecchetti mcecchetti requested review from timar and eszkadev October 28, 2023 07:51
@mcecchetti mcecchetti changed the title test patch for CI a11y: keep a11y disabled on executing cypress Calc tests Oct 28, 2023
@caolanm caolanm merged commit 09c18a4 into master Oct 29, 2023
7 checks passed
@caolanm caolanm deleted the private/marco/a11y/test_patch_for_CI branch October 29, 2023 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants