Skip to content
Back to pull request #5339

Unify token exchange for app managament client #18317

Sign in to view logs

Unify token exchange for app managament client

Unify token exchange for app managament client #18317

Triggered via pull request February 3, 2025 14:34
@isaacroldanisaacroldan
synchronize #5339
02-03-unify_token_exchange_for_app_managament_client
Status Success
Total duration 6m 50s
Artifacts 1

shopify-cli.yml

on: pull_request
[PR] Check OCLIF manifests & readme & docs
2m 28s
[PR] Check OCLIF manifests & readme & docs
[Manual] Test with Node ${{ inputs.node-version }} in ${{ inputs.os }}
0s
[Manual] Test with Node ${{ inputs.node-version }} in ${{ inputs.os }}
Matrix: main
Matrix: pr-acceptance-tests
Matrix: pr-platform-agnostic-bundle
Matrix: [PR] Check graphql-codegen has been run
Matrix: pr-platform-agnostic-knip
Matrix: pr-platform-agnostic-lint
Matrix: pr-platform-agnostic
Matrix: pr-platform-dependent
Matrix: pr-test-coverage
Fit to window
Zoom out
Zoom in

Annotations

2 warnings
[PR] Run knip with Node 18.20.3 in ubuntu-latest
knip stderr: The CJS build of Vite's Node API is deprecated. See https://vite.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.
packages/app/src/cli/utilities/developer-platform-client/app-management-client.ts#L427
[@typescript-eslint/prefer-nullish-coalescing] Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator.

Artifacts

Produced during runtime
Name Size
02-03-unify_token_exchange_for_app_managament_client--coverage-report
350 KB