search

martindstone / pagerduty-cli

A command line interface for PagerDuty
MIT License
91 stars 13 forks source link

`incident:assign -u` doesn't work as expected #18

Closed georgettica closed 2 years ago

georgettica commented 2 years ago

when looking at pd incident:assign -h you can clearly see two actions using the -u flag.

this stumped me for a while as I thought the userid I was providing was incorrect.

 -u, --assign_to_user_ids=assign_to_user_ids        User ID's to assign incidents to. Specify multiple times for multiple assignees.
  -u, --useauth=useauth                              Use the saved REST API token with this alias

maybe just remove the -u from the userauth? not sure how widely this is used

martindstone commented 2 years ago

Oops... thank you for reporting this! Would you be able to use the long form flag until I can get a fix in?

georgettica commented 2 years ago

I have a PR open on this with a link to this issue Thanks!

martindstone commented 2 years ago

hi @georgettica this should be fixed now (-u was a bad choice and conflicted with other things too) let me know if you have any trouble with it...