Back to pull request #29969

fix: Clicking on report link does not add external_link_clicked to si… #57274

Sign in to view logs

Triggered via pull request February 5, 2025 09:18

synchronize #29969

fix-signature-metrics

Status Success

Total duration 34m 44s

Artifacts 9

main.yml

on: pull_request

Wait for CircleCI workflow status / Wait for CircleCI workflow status

Build storybook / Build storybook

Build beta / Build beta

Matrix: Run tests / Unit tests

Matrix: Validate lavamoat policy webapp / Validate lavamoat policy webapp

Lint workflows / Lint workflows

Test lint shellcheck / Test lint shellcheck

Test lint / Test lint

Test lint changelog / Test lint changelog

Test lint lockfile / Test lint lockfile

Test yarn dedupe / Test yarn dedupe

Test deps depcheck / Test deps depcheck

Test storybook / Test storybook

Validate lavamoat allow scripts / Validate lavamoat allow scripts

Validate lavamoat policy build / Validate lavamoat policy build

Run tests / Webpack tests

Run tests / Integration tests

Test circular deps / Test circular dependencies

Test deps audit / Test deps audit

Runway / Runway

Publish prerelease / Publish prerelease

Run tests / Report coverage

All jobs completed

Log merge group failure / Log merge group failure

Annotations

10 warnings

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1265

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1286

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1306

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1390

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1406

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1429

Some tests seem to be commented

Test lint / Test lint: test/e2e/flask/solana/common-solana.ts#L91

'#fb9f18' Hex color values are not allowed. Consider using design tokens instead. For support reach out to the design system team #metamask-design-system on Slack

Test lint / Test lint: ui/components/app/detected-token/detected-token.js#L117

React Hook useEffect has a missing dependency: 'tokensListDetected'. Either include it or remove the dependency array

Test lint / Test lint: ui/components/app/name/name.stories.tsx#L129

Named exports should not use the name annotation if it is redundant to the name that would be generated by the export name

Test lint / Test lint: ui/components/app/snaps/snap-home-page/snap-home-renderer.js#L49

React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array

Artifacts

Produced during runtime

Name	Size
coverage-integration	821 KB
coverage-unit-1	1.05 MB
coverage-unit-2	1.05 MB
coverage-unit-3	1.05 MB
coverage-unit-4	1.06 MB
coverage-unit-5	1.04 MB
coverage-unit-6	1.05 MB
coverage-webpack	14.2 KB
lcov.info	325 KB