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

feat(workspace-plugin): make all react library generators work with both split project and old project setup #31142

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented Apr 23, 2024

Pre-requirements:

Previous Behavior

New Behavior

makes following generators work in both v9 package configuration ( old/current and new split library root-folder/library root-folder/stories:

  • react-library
  • react-component
  • cypress-component-configuration
  • bundle-size-configuration
  • prepare-initial-release

NOTE: once we migrate all packages those conditional (old) behaviours implemented will be removed from all changed generators in this PR ( except split-library-in-two generator )

Additional features:

  • react-library
    • once this is merged any new react library will start from scratch in split projects mode
  • cypress-component-configuration
    • properly add devDependencies when invoking
  • split-library-in-two
    • make cli log output configurable for better composability
    • enable execution on preview and compat packages
    • based on offline convo, test-ssr task will be moved to /stories project instead staying within /library as it's the right thing to do from domain and dependency graph POV.

Related Issue(s)

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 23, 2024

📊 Bundle size report

✅ No changes found

@Hotell Hotell changed the title Type checking perf/migration generator updates to work in split feat(workspace-plugin): make all react library generators work with both split project and old project setup Apr 23, 2024
Copy link

codesandbox-ci bot commented Apr 23, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@Hotell Hotell force-pushed the type-checking-perf/migration-generator-updates-to-work-in-split branch from 0817545 to e3ba234 Compare April 23, 2024 10:40
@Hotell Hotell force-pushed the type-checking-perf/migration-generator-updates-to-work-in-split branch from e3ba234 to 8c83190 Compare May 3, 2024 13:08
@Hotell Hotell marked this pull request as ready for review May 3, 2024 14:04
@Hotell Hotell requested a review from a team as a code owner May 3, 2024 14:04
@Hotell Hotell requested a review from marcosmoura May 7, 2024 12:13
@Hotell Hotell force-pushed the type-checking-perf/migration-generator-updates-to-work-in-split branch from 5492b9b to 4347358 Compare May 15, 2024 10:15
Hotell added 13 commits May 22, 2024 17:07
…ith new split /library /stories react projects
… with new split /library /stories react projects
…r work with new split /library /stories react projects
…y to avoid unhandled promises runtime issues
@Hotell Hotell force-pushed the type-checking-perf/migration-generator-updates-to-work-in-split branch from 4347358 to 9bc7084 Compare May 22, 2024 15:07
@Hotell Hotell enabled auto-merge (squash) May 22, 2024 15:07
@Hotell Hotell merged commit b153d70 into microsoft:master May 22, 2024
16 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants