This repository has been archived by the owner on Sep 30, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
web: migrate from yarn
to pnpm
#46143
Merged
+1,982
−35,851
Merged
Changes from all commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
05cace5
web: move global styles to `wildcard`
valerybugakov fd0b552
web: fix incorrect import path
valerybugakov ecd7a2b
Merge branch 'main' into vb/move-global-styles-to-wildcard
valerybugakov c182a5e
web: move global styles to `wildcard`
valerybugakov 04a46f7
web: test pnpm
valerybugakov fd4d746
chore: migrate yarn command to pnpm
gitstart-sourcegraph 1a35d26
Merge commit '2b0f6a5be8dda4dfc9686016bd67d60dc92a4816' of https://gi…
gitstart-sourcegraph 28d2cbf
chore: revert some changes + fix some CI check failing
gitstart-sourcegraph abdee47
[SG-45970] Add client packages to workspaces dependencies (#45981)
gitstart-sourcegraph 0809096
chore: revert some redundant changes
gitstart-sourcegraph 0541f28
chore: resolve conflicts
gitstart-sourcegraph 5c7960d
fix: build-ts
gitstart-sourcegraph 5fbbf04
fix: pnpm-install-with-retry.sh
gitstart-sourcegraph 62b57be
fix: esbuild
gitstart-sourcegraph 8903355
chore: update yarn usages in github action
gitstart-sourcegraph e6ee064
web: use the realpath for build config paths to support symlinks in w…
valerybugakov 5f56591
web: drop `yarn` related folders and configs
valerybugakov 83af6fe
web: update ignore files
valerybugakov 854ee1b
web: remove redundant pnpm usages
valerybugakov b01db04
[SG-45990] Migrate all the remaining yarn commands to pnpm (#46008)
gitstart-sourcegraph f459652
Merge branch 'vb/pnpm-test-2' into vb/pnpm-test-3
valerybugakov da1faeb
web: drop redundant dependencies
valerybugakov 66a72c5
web: prettier
valerybugakov 9ccfd0f
web: update Bazel workspace
valerybugakov ffbfc97
web: replace yarn
valerybugakov a226d50
web: replace yarn
valerybugakov a4da558
web: update `scip-typescript` to use `--pnpm-workspaces`
valerybugakov 78db32e
web: trigger the `scip-typescript` job
valerybugakov 93cbc20
web: update `dependency_decisions.yml`
valerybugakov eac5244
Merge branch 'main' into vb/pnpm-test-3
valerybugakov d9c324c
Revert "chore: revert some changes + fix some CI check failing"
valerybugakov ef6ac8e
Revert "chore: revert some redundant changes"
valerybugakov 3adae89
revert redundant changes
valerybugakov 74d7d7d
web: update `dependency_decisions.yml`
valerybugakov b1fcdff
web: fix TS build
valerybugakov e519abf
Merge branch 'main' into vb/pnpm-test-3
valerybugakov d5f0558
web: update `pnpm-lock.yaml`
valerybugakov e4fbcd5
vscode: migrate scripts to pnpm (#46194)
valerybugakov 316d16b
Merge branch 'main' into vb/pnpm-test-3
valerybugakov 861a2f3
web: more `yarn` —> `pnpm`
valerybugakov c18a855
web: disable hoisting
valerybugakov 9035e7e
web: add missing dependencies
valerybugakov 426357e
web: add missing dependencies
valerybugakov 68afdda
web: pnpm deduplicate
valerybugakov 6df89e9
web: update `web` fallbacks
valerybugakov d8de304
web: fix storybook build
valerybugakov aad643c
web: fix eslint
valerybugakov 63478f5
Merge branch 'main' into vb/pnpm-test-3
valerybugakov 5463031
web: update pnpm
valerybugakov b72f3d2
web: fix wildcard import
valerybugakov a926ca1
web: remove redundant import
valerybugakov 22f60cf
web: add `--frozen-lockfile` to `pnpm install` calls on CI
valerybugakov 0ef3809
web: drop the `--ignore-engines` flag
valerybugakov 6845ab7
web: remove redundant comment
valerybugakov ac7d314
web: update jest `transformIgnorePatterns`
valerybugakov a765b31
Merge branch 'main' into vb/pnpm-test-3
valerybugakov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,18 @@ | |
# rules_js lays out under Bazel (without a hidden node_modules/.pnpm/node_modules) | ||
hoist=false | ||
|
||
# We have to hoist storybook packages because storybok@6 is incompatible with pnpm and relies on | ||
# package hoisting for internal imports. We cannot migrate to storybook@7 because it's currently | ||
# in beta and multiple addons that we use are not yet compatible with the beta version. | ||
# The storybook@7 is fully compatible with pnpm, and once addons are updated, we can upgrade | ||
# and remove these hoist patterns. For more context, see: | ||
# https://github.com/storybookjs/storybook/issues/13428#issuecomment-1368062647 | ||
public-hoist-pattern[]=*storybook* | ||
public-hoist-pattern[]=global | ||
|
||
# We have to hoist eslint packages to use them in the `.eslintrc` config. | ||
public-hoist-pattern[]=*eslint* | ||
Comment on lines
+15
to
+16
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is only required when Storybook packages are hoisted. We can remove both patterns after upgrading Storybook to v7. |
||
|
||
# npm_translate_lock with the yarn.lock file will complain about missing/incorrect peer dependencies. | ||
# Temporarily ignore warnings until we switch to pnpm and can user pnpm.packageExtensions to fix them. | ||
# https://pnpm.io/package_json#pnpmpackageextensions | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To avoid hard coding the version in the github workflow, I've used this action before. Maybe we can reuse it here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find, I love that. Let's do it in a follow-up PR!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh nice, that's awesome!