Skip to content

feat(tools): implement migrate-scripts-to-pkg generator#26129

Closed
Hotell wants to merge 2 commits intomicrosoft:masterfrom
Hotell:hotell/build/scripts/ts-solution-config-v15
Closed

feat(tools): implement migrate-scripts-to-pkg generator#26129
Hotell wants to merge 2 commits intomicrosoft:masterfrom
Hotell:hotell/build/scripts/ts-solution-config-v15

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented Dec 29, 2022

New Behavior

Implements nx generator that will migrate @fluentui/scripts to separate packages and update devDeps in whole monorepo

Related Issue(s)

TODO:

  • if project has gulpfile add scripts-gulpfile
  • don't add script packages root package.json if they are used in folder that is not a package (/.storybook for example)
  • eslint: turn off import-extranous deps -> set workspace root
  • eslint: turn off @fluentui/max-len
  • eslint: turn off import/no-self-import for index.d.ts
  • tsconfig.jsons have wrong globs generated
  • remove fluentui/scripts from all package.json ( currently if there is no direct import in package we won't process its package.json )
  • add fluentui/scripts-api-extractor devDep to packages that use api-extractor
  • update plop templates to include correct scripts devDeps (should be only api-extractor, tasks )

@Hotell Hotell force-pushed the hotell/build/scripts/ts-solution-config-v15 branch from 4b35d68 to b561ad0 Compare December 29, 2022 18:01
@Hotell Hotell changed the title Hotell/build/scripts/ts solution config v15 feat(tools): implement migrate-scripts-to-pkg generator Dec 29, 2022
@fabricteam
Copy link
Collaborator

fabricteam commented Dec 29, 2022

📊 Bundle size report

🤖 This report was generated against 26f0364b3837056ee8e0df42a7932c298c68290e

@size-auditor
Copy link

size-auditor bot commented Dec 29, 2022

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 26f0364b3837056ee8e0df42a7932c298c68290e (build)

@Hotell Hotell force-pushed the hotell/build/scripts/ts-solution-config-v15 branch from b561ad0 to 9851d42 Compare December 30, 2022 16:54
@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 30, 2022

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.

Latest deployment of this branch, based on commit f86a621:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@Hotell Hotell force-pushed the hotell/build/scripts/ts-solution-config-v15 branch from 9851d42 to 9280c2e Compare January 4, 2023 11:38
@Hotell Hotell force-pushed the hotell/build/scripts/ts-solution-config-v15 branch 7 times, most recently from f4e0a46 to 3c73e78 Compare January 5, 2023 16:47
@Hotell Hotell force-pushed the hotell/build/scripts/ts-solution-config-v15 branch 2 times, most recently from 3bf8f8f to 3c73e78 Compare January 9, 2023 14:10
@Hotell Hotell force-pushed the hotell/build/scripts/ts-solution-config-v15 branch from 3c73e78 to f86a621 Compare January 9, 2023 14:11
@Hotell
Copy link
Contributor Author

Hotell commented Jan 18, 2023

this wont be merged, as it adds lot of code that we don't have to/need to maintain, as this was only 1 time migration. It will remain in pull request for future education purposes 🙌.

@Hotell Hotell closed this Jan 18, 2023
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.

3 participants