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
Added a Meaningful error msg for control server port taken #3204
Added a Meaningful error msg for control server port taken #3204
Conversation
test/config/activate_control_app.rb
Outdated
|
||
app do |env| | ||
[200, {}, ["OK"]] | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this file added? Can't see in the diff that it is being used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment - change what class we're rescuing.
Also, this branch is out-of-date with the base branch |
You can do either, but I would recommend squashing the commits if you go for rebase. When merged, everything will be squashed anyway. |
@nateberkopec this can be merged now? |
Thank you @dhavalsingh ! |
Description
Please describe your pull request. Thank you for contributing! You're the best.
Your checklist for this pull request
[ci skip]
to the title of the PR.#issue
" to the PR description or my commit messages.Closes issue 3204
Might close issue #2894 too.