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

test: lighthouse checks on e2e #1425

Merged
merged 1 commit into from
Mar 6, 2024
Merged

test: lighthouse checks on e2e #1425

merged 1 commit into from
Mar 6, 2024

Conversation

fpaul-1A
Copy link
Contributor

@fpaul-1A fpaul-1A commented Feb 29, 2024

Proposed change

The goal is to prevent regressions regarding accessibility on the showcase app
image

Related issues

  • 🐛 Fixes #(issue)
  • 🚀 Feature #(issue)

Copy link

nx-cloud bot commented Feb 29, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit aebb78c. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@fpaul-1A fpaul-1A force-pushed the test/e2e-lighthouse branch 2 times, most recently from 8ee42f1 to 0545a28 Compare March 5, 2024 11:15
@fpaul-1A fpaul-1A changed the title [draft] test: lighthouse checks on e2e test: lighthouse checks on e2e Mar 5, 2024
@fpaul-1A fpaul-1A marked this pull request as ready for review March 5, 2024 11:17
@fpaul-1A fpaul-1A requested a review from a team as a code owner March 5, 2024 11:17
@fpaul-1A fpaul-1A force-pushed the test/e2e-lighthouse branch 3 times, most recently from ae6e026 to 8e5bdd2 Compare March 5, 2024 15:00
@@ -230,12 +230,14 @@
"js-yaml": "^4.1.0",
"jsonc-eslint-parser": "~2.4.0",
"jsonpath-plus": "^8.0.0",
"lighthouse": "9.6.8",
Copy link
Member

Choose a reason for hiding this comment

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

why frozen versions for the new deps?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

newer versions don't seem to support CommonJS abhinaba-ghosh/playwright-lighthouse#51

@fpaul-1A fpaul-1A added this pull request to the merge queue Mar 6, 2024
Merged via the queue into main with commit a9e7643 Mar 6, 2024
28 checks passed
@fpaul-1A fpaul-1A deleted the test/e2e-lighthouse branch March 6, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants