Skip to content

task: audit yarn resolutions – moment#18927

Merged
toufali merged 1 commit intomainfrom
remove-moment-resolution
May 28, 2025
Merged

task: audit yarn resolutions – moment#18927
toufali merged 1 commit intomainfrom
remove-moment-resolution

Conversation

@toufali
Copy link
Member

@toufali toufali commented May 28, 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 moment

Issue that this pull request solves

Closes: FXA-11765

Other information (Optional)

I tested before and after resolution removal: all moment dependencies are at 2.30.1 which satisfy the original resolution.

@toufali toufali marked this pull request as ready for review May 28, 2025 00:25
@toufali toufali requested a review from a team as a code owner May 28, 2025 00:25
@chenba
Copy link
Contributor

chenba commented May 28, 2025

I expect a yarn.lock change. Did removing the resolution have no effect on the actual installed version(s) of moment.js?

@toufali
Copy link
Member Author

toufali commented May 28, 2025

I expect a yarn.lock change. Did removing the resolution have no effect on the actual installed version(s) of moment.js?

Correct @chenba – similar to #18926 I think this resolution had been resolved, or the resolution was noop in the first place. Running yarn why moment before the removal satisfied the resolution conditions, just as after removal. Running yarn install has no effect.

@toufali toufali merged commit c04f207 into main May 28, 2025
19 checks passed
@toufali toufali deleted the remove-moment-resolution branch May 28, 2025 17:45
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