The argument ordering question #11

Closed
opened 2021-07-21 21:21:38 +00:00 by decentral1se · 2 comments
Owner

I was trying to wire up urfave to use abra recipe <recipe> version ordering but as I have discovered, I have no idea how to do this.

@3wordchant mentioned a discussion with @roxxers about this and I see some rumours of an outcome in https://git.autonomic.zone/coop-cloud/abra/issues/189.

I am personally fine with moving to abra recipe version <recipe> ordering and moving all non-global flags/args to the end of the command order (as I believe @knoflook was suggesting?).

I am wary of moving into "making new things" in this port while we're still trying to get to feature parity but this one seems like a fine shortcut to take since parsing the original ordering looks tricky & hard to do.

Thoughts?

I was trying to wire up urfave to use `abra recipe <recipe> version` ordering but as I have discovered, I have no idea how to do this. @3wordchant mentioned a discussion with @roxxers about this and I see some rumours of an outcome in https://git.autonomic.zone/coop-cloud/abra/issues/189. I am personally fine with moving to `abra recipe version <recipe>` ordering and moving all non-global flags/args to the end of the command order (as I believe @knoflook was suggesting?). I am wary of moving into "making new things" in this port while we're still trying to get to feature parity but this one seems like a fine shortcut to take since parsing the original ordering looks tricky & hard to do. Thoughts?
Author
Owner

Some logs attached...

Some logs attached...
decentral1se added the
question
help wanted
labels 2021-07-22 11:09:49 +00:00
Author
Owner

Went with abra recipe versions <recipe> in 6eee02d90a, hope that is OK! Seems handiest now that the lib does that by default and some folks are asking for it. Will close off for now.

Went with `abra recipe versions <recipe>` in https://git.coopcloud.tech/coop-cloud/go-abra/commit/6eee02d90abe6018fa9a62d6e1e960d3fabe37c7, hope that is OK! Seems handiest now that the lib does that by default and some folks are asking for it. Will close off for now.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#11
No description provided.