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

Support chaining commands on CLI #13

Closed
donmccurdy opened this issue Jan 15, 2019 · 4 comments · Fixed by #819
Closed

Support chaining commands on CLI #13

donmccurdy opened this issue Jan 15, 2019 · 4 comments · Fixed by #819
Labels
feature New enhancement or request package:cli
Milestone

Comments

@donmccurdy
Copy link
Owner

gltf-transform \
  [ prune ] \
  [ atlas --size 1024 ] \
  [ ao --samples 500 ]

https://github.com/SamVerschueren/listr

@donmccurdy donmccurdy added the feature New enhancement or request label Jan 15, 2019
@donmccurdy
Copy link
Owner Author

this could make sense for the core library too...

container
  .transform(ao({samples: 100}))
  .transform(atlas({resolution: 256}));
container.pipe(
  ao({samples: 100}),
  atlas({resolution: 256})
);

@donmccurdy donmccurdy added this to the v0.2.0 milestone Apr 11, 2020
@donmccurdy
Copy link
Owner Author

container.transform(...) is will work. I don't feel inclined to ditch Caporal for the CLI, and it doesn't seem to have an easy way to support chaining, so no CLI chaining for now.

@donmccurdy
Copy link
Owner Author

Idea for doing this with Caporal would be to use a pseudo-command syntax:

gltf-transform a+b+c in.glb out.glb

All commands would use default options. Detecting this when the CLI program starts we can dynamically inject the new compound command.

@donmccurdy donmccurdy reopened this Aug 14, 2022
@donmccurdy donmccurdy modified the milestones: v0.2, Backlog Aug 14, 2022
@donmccurdy donmccurdy changed the title [cli] chaining mode Support chaining commands on CLI Jan 13, 2023
@donmccurdy
Copy link
Owner Author

Alternatively... maybe a new uber-command that does an appropriate full pipeline for you, with a few options but not necessarily everything. Chaining is kind of less necessary with the proposal for #791 anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New enhancement or request package:cli
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant