node-gh / gh

(DEPRECATED) GitHub CLI made with NodeJS. Use the official https://cli.github.com/ instead.
http://nodegh.io
Other
1.71k stars 217 forks source link

😻 Updates 🎉 #656

Open protoEvangelion opened 5 years ago

protoEvangelion commented 5 years ago

If you are interested on gh updates, you can subscribe to this issue.

New v2 Features

image

Node Versions

Issues

Milestones

Pull Requests

Repo

Hooks

House Cleaning

About myself 👋

Hi there, I’m Ryan! My goal is to improve your dev life by increasing your productivity. If you run into any issues, or would like to contribute, feel free to reach out to me and it would be my joy to help!

drewbrokke commented 5 years ago

Nice work! I don’t fully understand removing the alias support though. I may be missing the obvious answer, but how would I go about replacing that functionality with a bash alias? So if I aliased a username ‘reallylomgusername’ to ‘user’, with the aliasing i’d just type gh pr —submit user. Now that it’s removed, the suggested replacement would to use standard shell expansion? So I’d type gh pr —submit $user?

protoEvangelion commented 5 years ago

Yes. When we were collecting telemetry, that didn't seem like something that was used. Additionally, most people I have seen using this tool, just use their own bash aliases. So I didn't want to reproduce what the shell can already do well ;)

thektan commented 5 years ago

I use aliases as well. 😢 Was there overhead maintaining that feature? I'm personally not a fan of using the $ character, but I suppose I can adapt.

protoEvangelion commented 5 years ago

Thanks for chiming in @thektan . I am open to adding it back. How else do you do aliases in every other app though? For instance, I add github user aliases in my .zshrc and then can use them in any cli program. Shell aliases are more powerful and portable.

thektan commented 5 years ago

@protoEvangelion I don't use aliases with other apps; the closest would probably be git remotes. My aliases are typically acronyms of complete commands (alias yb=yarn build). But if it's more standard to incorporate commands using variables with $ maybe it's for the better to start using that more.

Although another benefit for having aliases specific to an app would be allowing for shorter aliases like m for myUser whereas in another app m could be used for master.

But ya, if it's more to maintain no worries! I appreciate the time you've taken to maintain this repo. 💯