Skip to content

task: audit yarn resolutions – browserid-crypto#18949

Merged
toufali merged 1 commit intomainfrom
remove-resolution-browserid-crypto
Jun 3, 2025
Merged

task: audit yarn resolutions – browserid-crypto#18949
toufali merged 1 commit intomainfrom
remove-resolution-browserid-crypto

Conversation

@toufali
Copy link
Member

@toufali toufali commented Jun 2, 2025

Because

  • Yarn resolutions should be used as last resort
  • It doesn't look like this particular resolution does anything

This pull request

  • removes the resolution for browserid-crypto

Issue that this pull request solves

Closes: FXA-11790

Other information (Optional)

It doesn't appear that browserid-crypto is used anymore – neither directly or as a transitive dependency. Running yarn why browserid-crypto before and after removal returns nothing, and a lockfile is not generated after yarn install.

@toufali toufali requested a review from a team as a code owner June 2, 2025 21:32
@toufali toufali merged commit 6bc489e into main Jun 3, 2025
19 checks passed
@toufali toufali deleted the remove-resolution-browserid-crypto branch June 3, 2025 16:20
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