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

Add option '--port' to 'rails server' alias. #4486

Merged
merged 1 commit into from
Oct 14, 2015
Merged

Add option '--port' to 'rails server' alias. #4486

merged 1 commit into from
Oct 14, 2015

Conversation

gfvcastro
Copy link
Contributor

No description provided.

robbyrussell added a commit that referenced this pull request Oct 14, 2015
Add option '--port' to 'rails server' alias.
@robbyrussell robbyrussell merged commit 30d7a66 into ohmyzsh:master Oct 14, 2015
@KevinBongart
Copy link
Contributor

Hey @gfvcastro & @robbyrussell, thanks for keeping this Rails plugin alive!

Just wanted to provide some quick feedback: I used to run my specs by typing rsp then pressing tab and it would be autocompleted into rspec. With this alias, I can't anymore :(

Sure, it's an extremely minor annoyance, perfectly comparable to this:

But I'm wondering whether such a specific command needs to be added as one of the default aliases for all Rails developers who use OMZ (I have a lot of aliases that I keep in my own .zshrc).

If other people start having the same issue, maybe we could reconsider?

@gfvcastro
Copy link
Contributor Author

@KevinBongart: Sorry. Oh... You are right...

@KevinBongart
Copy link
Contributor

@gfvcastro It's alright! Since then, I just adapted my workflow by typing one more letter :)

nunogt pushed a commit to nunogt/oh-my-zsh that referenced this pull request Jan 25, 2016
…t-option

Add option '--port' to 'rails server' alias.
@gfvcastro gfvcastro deleted the alias-rails-server-port-option branch August 13, 2016 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants