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: Support pnpm workspaces #2058

Merged
merged 4 commits into from
Sep 12, 2023
Merged

Conversation

Jason3S
Copy link
Contributor

@Jason3S Jason3S commented Aug 30, 2023

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #2057 🦕
fixes #1587

@Jason3S Jason3S requested review from a team as code owners August 30, 2023 14:58
@google-cla
Copy link

google-cla bot commented Aug 30, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@product-auto-label product-auto-label bot added the size: xl Pull request size is extra large. label Aug 30, 2023
@Jason-Rev
Copy link

@chingor13,

Thank you for running the tests.

npm ERR! notsup Required: {"node":">=14.17.0","npm":">=8.0.0"}
npm ERR! notsup Actual:   {"npm":"6.14.18","node":"14.21.3"}

It looks like lerna-lite requires npm 8+. What is the long term npm support plan for release-please?

Kind regards,
Jason

@chingor13
Copy link
Contributor

Thank you for this and it looks like a reasonable change.

It looks like lerna-lite requires npm 8+. What is the long term npm support plan for release-please?

We currently support node 14+ (which is already EoL). I would like to release a new version that targets 18+, but we run this in a bot framework that also needs to be upgraded to node 18+ which I am currently looking into.

@chingor13 chingor13 added this to the v16 (node 18+) milestone Sep 7, 2023
@chingor13 chingor13 merged commit 13cba14 into googleapis:main Sep 12, 2023
10 checks passed
@Jason3S Jason3S deleted the dev-lerna-lite branch September 13, 2023 05:39
@Jason3S
Copy link
Contributor Author

Jason3S commented Sep 13, 2023

@chingor13,

Thank you.

kalosisz added a commit to kalosisz/release-please that referenced this pull request Sep 19, 2023
chore(deps): update actions/checkout action to v4 (googleapis#2060)

docs: check `autorelease: pending` or `autorelease: triggered` in an old PR (googleapis#1930)

* docs: check `autorelease: pending` or `autorelease: triggered` in an old PR

In googleapis/java-logging-logback#1050, I had to remove the "autorelease: pending " label to create a release pull request.

* Update README.md

fix mismerge

---------

Co-authored-by: Jeff Ching <chingor@google.com>

test: also test on node 18 (googleapis#2063)

* test: also test on node 18

* ci: enable no-experimental-fetch for tests

* ci: set NODE_OPTIONS for node 18

* set NODE_OPTIONS for windows build

* set NODE_OPTIONS for windows build

ci: only require node 18 tests (googleapis#2065)

feat!: require node 18+ (googleapis#2069)

fix(deps): bump semver dependency version (googleapis#2068)

feat: Support pnpm workspaces (googleapis#2058)

fix(deps): replace lerna dependency with lerna-lite

Co-authored-by: Jeff Ching <chingor@google.com>

fix(deps): upgrade http-proxy-agent to v7 (googleapis#2071)

fix(deps): upgrade https-proxy-agent to v7

feat: fallback to root package version if package ignores github release (googleapis#1935)

Co-authored-by: Jeff Ching <chingor@google.com>

chore(deps): update dependency sinon to v16 (googleapis#2079)

feat: Use default updaters based on file extension (googleapis#2072)

* feat: Use default updaters based on file extension

* use CompositeUpdater

* improve things

---------

Co-authored-by: Jeff Ching <chingor@google.com>

chore(main): release 16.0.0 (googleapis#2070)

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

config

docs
kalosisz added a commit to kalosisz/release-please that referenced this pull request Sep 19, 2023
chore(deps): update actions/checkout action to v4 (googleapis#2060)

docs: check `autorelease: pending` or `autorelease: triggered` in an old PR (googleapis#1930)

* docs: check `autorelease: pending` or `autorelease: triggered` in an old PR

In googleapis/java-logging-logback#1050, I had to remove the "autorelease: pending " label to create a release pull request.

* Update README.md

fix mismerge

---------

Co-authored-by: Jeff Ching <chingor@google.com>

test: also test on node 18 (googleapis#2063)

* test: also test on node 18

* ci: enable no-experimental-fetch for tests

* ci: set NODE_OPTIONS for node 18

* set NODE_OPTIONS for windows build

* set NODE_OPTIONS for windows build

ci: only require node 18 tests (googleapis#2065)

feat!: require node 18+ (googleapis#2069)

fix(deps): bump semver dependency version (googleapis#2068)

feat: Support pnpm workspaces (googleapis#2058)

fix(deps): replace lerna dependency with lerna-lite

Co-authored-by: Jeff Ching <chingor@google.com>

fix(deps): upgrade http-proxy-agent to v7 (googleapis#2071)

fix(deps): upgrade https-proxy-agent to v7

feat: fallback to root package version if package ignores github release (googleapis#1935)

Co-authored-by: Jeff Ching <chingor@google.com>

chore(deps): update dependency sinon to v16 (googleapis#2079)

feat: Use default updaters based on file extension (googleapis#2072)

* feat: Use default updaters based on file extension

* use CompositeUpdater

* improve things

---------

Co-authored-by: Jeff Ching <chingor@google.com>

chore(main): release 16.0.0 (googleapis#2070)

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

config

docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: xl Pull request size is extra large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider moving to lerna-lite plugin node_workspace doesn't work with pnpm?
3 participants