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

DRAFT PNPM sync with Master #4955

Closed
wants to merge 150 commits into from
Closed

Conversation

smouillour
Copy link
Contributor

What is the problem this PR is trying to solve?

What is the chosen solution to this problem?

Please check if the PR fulfills these requirements

  • The PR have used yarn changeset to a request a release from the CI if wanted.
  • The PR commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features) And non reg done before need review
  • Docs have been added / updated (for bug fixes / features)
  • Related design / discussions / pages (not in jira), if any, are all linked or available in the PR

[ ] This PR introduces a breaking change

smouillour and others added 30 commits October 12, 2023 11:25
* remove console.log

* fix ci test

* fix cache ci test

* fix gha + jest config

* fix node_modules path, upgrade babel/core, keep jest only in scripts-core

* chore: pnpm-deduplicate

* add @talend/scripts-config-storybook-lib in every package having storybook

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@smouillour smouillour closed this Apr 3, 2024
@smouillour smouillour deleted the smouillour/fix/TDOPS-5327-pnpm-7 branch April 3, 2024 16:54
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.

2 participants