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

refactor(cli-utils): Refactor CLI entrypoint and remove exports #197

Merged
merged 16 commits into from
Apr 13, 2024

Conversation

kitten
Copy link
Member

@kitten kitten commented Apr 13, 2024

Summary

This basically aims to achieve three things:

  • Remove exports so we can revisit: RFC: Exposing the generate-schema and generate-output actions from cli-utils as Node.js API's #134 (comment)
    • I think we can either abstract into @gql.tada/internal (pretty complex tbf)
    • or; we can just provide an api/ folder in cli-utils and make proper variants, so we still have some options in altering CLI outputs
  • Replace sade with clipanion to split up the commands and have some more QoL features available and abstracted away (sub commands, validation)
  • Add/Fix some more CLI options and add some stdout pipe options

This doesn't yet:

  • Remove warnings, swapping them out with errors and null/undefined return values here and there
  • Update the CLI outputs with new colorisers
  • Investigate/Fix why init has no styling
  • Resolve some todos and add extra CLI options
  • Add a custom help command / output

Set of changes

Copy link

changeset-bot bot commented Apr 13, 2024

🦋 Changeset detected

Latest commit: 28cf9dc

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

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

@kitten kitten merged commit b1b8ba6 into main Apr 13, 2024
2 checks passed
@kitten kitten deleted the feat/switch-to-clipanion branch April 13, 2024 19:07
@github-actions github-actions bot mentioned this pull request Apr 13, 2024
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.

None yet

2 participants