-
Notifications
You must be signed in to change notification settings - Fork 12
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
Default to rollups
when --framework
is not defined
#173
Comments
rollups
when --framwork
is not definedrollups
when --framework
is not defined
This can be hard to do and also can be hard for the user to understand. Now, what should be printed with the I don't know if this is easy or possible to do with commander. |
Thinking better about this, maybe we should deprecate the root As far as I understood, the goal to have a What if we only support This is related to the PR #174 and #175 @tuler let's define a better scope for this, so that the coprocessor integration to the CLI can move forward. |
The problem of having a root command is that each subcommand have different options, which are not “transferred” to the root command. |
So I'm proposing to not have a And other command we see necessary. For me, we only need root command for
Those depends on
Everything else, like |
📚 Context
With the addition of the
cartesi rollups <subdommands>
command preparing for the support ofcartesi coprocessor <subcommands>
we should default to therollups
command when rollups or coprocessor are not defined.✔️ Solution
We should add a
--framework
CLI argument that will control ifrollups
orcoprocessor
will be used.The text was updated successfully, but these errors were encountered: