Skip to content

Conversation

@lorisleiva
Copy link
Member

This is causing renderer updates to unnecessarily bump core Codama libraries.

This is causing renderer updates to unnecessarily bump core Codama libraries.
@changeset-bot
Copy link

changeset-bot bot commented May 12, 2025

🦋 Changeset detected

Latest commit: 016cdc7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages
Name Type
codama Minor
@codama/cli Minor
@codama/errors Minor
@codama/node-types Minor
@codama/nodes Minor
@codama/validators Minor
@codama/visitors-core Minor
@codama/visitors Minor
@codama/dynamic-codecs Patch
@codama/dynamic-parsers Patch
@codama/nodes-from-anchor Patch
@codama/renderers-core Patch
@codama/renderers-js-umi Patch
@codama/renderers-js Patch
@codama/renderers-rust Patch
@codama/renderers-vixen-parser Patch
@codama/renderers Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@lorisleiva lorisleiva marked this pull request as ready for review May 12, 2025 09:43
@lorisleiva lorisleiva requested a review from febo May 12, 2025 10:18
Copy link
Contributor

@febo febo left a comment

Choose a reason for hiding this comment

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

👌

@lorisleiva lorisleiva merged commit 23f3795 into main May 12, 2025
4 checks passed
@lorisleiva lorisleiva deleted the loris/decouple-cli-from-codama branch May 12, 2025 10:52
@github-actions github-actions bot mentioned this pull request May 12, 2025
lorisleiva added a commit that referenced this pull request Aug 18, 2025
This PR adds the `codama` CLI back to the main library so it is available when consumers install `codama` — instead of having to additionally install `@codama/cli`.

This change was reverted before (See <#596>) because `@codama/cli` contained various dependencies (such as renderers) that would then be linked with the main library when versioning package. This caused issues such as: when bumping a renderer package, the node package would need to also be bumped. This limitation is now gone since these additionally dependencies have been removed throughout the PR stack that this PR is part of.
TimothyBarrett3860TimothyBarrett added a commit to TimothyBarrett3860TimothyBarrett/codama that referenced this pull request Oct 4, 2025
This PR adds the `codama` CLI back to the main library so it is available when consumers install `codama` — instead of having to additionally install `@codama/cli`.

This change was reverted before (See <codama-idl/codama#596>) because `@codama/cli` contained various dependencies (such as renderers) that would then be linked with the main library when versioning package. This caused issues such as: when bumping a renderer package, the node package would need to also be bumped. This limitation is now gone since these additionally dependencies have been removed throughout the PR stack that this PR is part of.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants