Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better support for developing plugins from within yarn 2+ workspaces (include __virtual folder in build) #3109

Merged
merged 1 commit into from
Jul 25, 2022

Conversation

garrettjstevens
Copy link
Collaborator

In yarn 2+ workspaces, packages that are shared in the workspace don't show up under node_modules, they show up under a __virtual__ folder. This adds handling of that to the plugin rollup config, so that shared workspace packages are processed by rollup correctly.

This is for shared packages in yarn 4 workspaces, where the resolved
path does not contain node_modules, but they still need to be treated as
external modules.
@garrettjstevens garrettjstevens added the bug Something isn't working label Jul 25, 2022
@garrettjstevens garrettjstevens self-assigned this Jul 25, 2022
@codecov
Copy link

codecov bot commented Jul 25, 2022

Codecov Report

Merging #3109 (8ca216d) into main (18290ec) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3109      +/-   ##
==========================================
- Coverage   58.91%   58.89%   -0.02%     
==========================================
  Files         640      640              
  Lines       28843    28843              
  Branches     6999     6999              
==========================================
- Hits        16992    16987       -5     
- Misses      11469    11474       +5     
  Partials      382      382              
Impacted Files Coverage Δ
...svg/src/SvgFeatureRenderer/components/Segments.tsx 85.71% <0.00%> (-7.15%) ⬇️
...src/SvgFeatureRenderer/components/FeatureLabel.tsx 80.55% <0.00%> (-5.56%) ⬇️
...gins/svg/src/SvgFeatureRenderer/components/util.ts 91.83% <0.00%> (-4.09%) ⬇️
...FeatureRenderer/components/ProcessedTranscript.tsx 87.32% <0.00%> (-1.41%) ⬇️
packages/core/util/layouts/GranularRectLayout.ts 86.86% <0.00%> (-0.43%) ⬇️
...ments/src/SNPCoverageAdapter/SNPCoverageAdapter.ts 58.00% <0.00%> (+0.66%) ⬆️
plugins/alignments/src/BamAdapter/BamAdapter.ts 72.97% <0.00%> (+0.90%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us.

@cmdcolin cmdcolin merged commit e83b704 into main Jul 25, 2022
@cmdcolin cmdcolin deleted the rollup_yarn_4_workspace branch July 25, 2022 22:51
@cmdcolin cmdcolin changed the title Add __virtual__ to rollup files to process Better support for developing plugins from within yarn 2+ workspaces (include __virtual folder in build) Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants