Skip to content

Conversation

@lorisleiva
Copy link
Member

@lorisleiva lorisleiva commented 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.

@changeset-bot
Copy link

changeset-bot bot commented Aug 18, 2025

🦋 Changeset detected

Latest commit: 2515f4b

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

This PR includes changesets to release 17 packages
Name Type
codama Patch
@codama/cli Patch
@codama/errors Patch
@codama/node-types Patch
@codama/nodes Patch
@codama/validators Patch
@codama/visitors-core Patch
@codama/visitors Patch
@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 August 18, 2025 14:35
@lorisleiva lorisleiva force-pushed the 08-18-check_missing_dependencies_before_run_cli_command branch from 080370c to 43121c2 Compare August 18, 2025 14:37
@lorisleiva lorisleiva force-pushed the 08-18-include_cli_in_main_codama_library branch 2 times, most recently from d713520 to a6164c2 Compare August 18, 2025 14:39
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.

🙌

Copy link
Member Author

lorisleiva commented Aug 18, 2025

Merge activity

  • Aug 18, 2:55 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 18, 3:25 PM UTC: Graphite rebased this pull request as part of a merge.
  • Aug 18, 3:26 PM UTC: @lorisleiva merged this pull request with Graphite.

@lorisleiva lorisleiva changed the base branch from 08-18-check_missing_dependencies_before_run_cli_command to graphite-base/777 August 18, 2025 15:21
@lorisleiva lorisleiva changed the base branch from graphite-base/777 to main August 18, 2025 15:23
@lorisleiva lorisleiva force-pushed the 08-18-include_cli_in_main_codama_library branch from a6164c2 to 2515f4b Compare August 18, 2025 15:24
@lorisleiva lorisleiva merged commit 14e1614 into main Aug 18, 2025
3 checks passed
@lorisleiva lorisleiva deleted the 08-18-include_cli_in_main_codama_library branch August 18, 2025 15:26
@github-actions github-actions bot mentioned this pull request Aug 18, 2025
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