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

Fix Lerna build of all packages #1204

Merged
merged 5 commits into from
Jan 8, 2024
Merged

Fix Lerna build of all packages #1204

merged 5 commits into from
Jan 8, 2024

Conversation

literat
Copy link
Collaborator

@literat literat commented Jan 5, 2024

Description

  • lerna should determine paralelization based on dependencies or
    devDependencies, but this do not work 🤷
  • we can use yarn lerna run build --concurrency 1 to disable this
    but it is slow and takes a long time
  • design-tokens must be build first so I have splitted up the web*
    packages to be build after the rest of the packages
  • problem can be test by simply running yarn lerna run build --scope=@lmc-eu/spirit-design-tokens --scope=@lmc-eu/spirit-web-react

Additional context

https://stackoverflow.com/questions/67154137/how-to-run-lerna-sequentially

Issue reference


Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Follow the PR Title/Commit Message Convention.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

  * this will work when running `yarn lerna run build --scope
    @lmc-eu/spirit-design-tokens`
  * lerna should determine paralelization based on dependencies or
    devDependencies, but this do not work 🤷
  * we can use `yarn lerna run build --concurrency 1` to disable this
    but it is slow and takes a long time
  * `design-tokens` must be build first so I have splitted up the `web*`
    packages to be build after the rest of the packages
  * problem can be test by simply running `yarn lerna run build
    --scope=@lmc-eu/spirit-design-tokens
--scope=@lmc-eu/spirit-web-react`
Copy link

netlify bot commented Jan 5, 2024

Deploy Preview for spirit-design-system-storybook canceled.

Name Link
🔨 Latest commit cc643b0
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/6597f45a5479090008805f2d

@github-actions github-actions bot added the maintenance Changes to the build process or auxilary tools and libraries or other maintenance label Jan 5, 2024
Copy link

netlify bot commented Jan 5, 2024

Deploy Preview for spirit-design-system-demo canceled.

Name Link
🔨 Latest commit cc643b0
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-demo/deploys/6597f45ad6135d000888cf78

Copy link

netlify bot commented Jan 5, 2024

Deploy Preview for spirit-design-system-validations canceled.

Name Link
🔨 Latest commit cc643b0
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-validations/deploys/6597f45ac0402300080116c5

Copy link

netlify bot commented Jan 5, 2024

Deploy Preview for spirit-design-system-react canceled.

Name Link
🔨 Latest commit cc643b0
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-react/deploys/6597f45ac4d873000839fa61

Copy link
Contributor

github-actions bot commented Jan 5, 2024

Coverage Status

coverage: 70.462% (-9.2%) from 79.697%
when pulling cc643b0 on chore/nx-lerna-build
into cbe878b on main.

@coveralls
Copy link

Coverage Status

coverage: 79.697%. remained the same
when pulling cc643b0 on chore/nx-lerna-build
into cbe878b on main.

Copy link
Contributor

@pavelklibani pavelklibani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working for me, thank you!

@literat literat merged commit 6335a67 into main Jan 8, 2024
23 checks passed
@literat literat deleted the chore/nx-lerna-build branch January 8, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Changes to the build process or auxilary tools and libraries or other maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants