-
Notifications
You must be signed in to change notification settings - Fork 178
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
Docker_image.rst: remove pip install instruction #489
Conversation
LGTM. |
ack f37de42 |
@gitmate-bot rebase |
This commit makes sure that the `pip install -r requirements.txt` will no longer appear in Docker_image.rst. It is not necessary with a docker image. Closes coala#486
Hey! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost! |
Automated rebase was successful! |
unack |
Please rewrite commit message. "This commit makes sure that the ..." is too wordy - that could be the first words of every commit message, but it is not ... it would be redundant to start every commit message like that. Describe briefly what is in the commit, and why. |
unack 40329da |
(and rebase your commit) |
@jayvdb I have updated my comment. How do I do rebase? |
http://docs.coala.io/en/latest/Users/Docker_Image.html#coala-on-gitlab-ci mentions invokingpip install -r requirements.txt. It is unnecessary and the documentation has been updated to show the changes. Closes coala#486
three commits instead of one. |
Why closed this PR? Btw this happened with me also :P |
Yes, I will do it
…On Nov 9, 2017 13:33, "GitMate" ***@***.***> wrote:
Assigned #489 <#489> to
@vipuldcoder <https://github.com/vipuldcoder>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#489 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AV-mXDGga6TiuJruJuwpDppHFUpRZDYeks5s0rHLgaJpZM4QSHjA>
.
|
Docker_image.rst: remove pip install instruction
http://docs.coala.io/en/latest/Users/Docker_Image.html#coala-on-gitlab-ci mentions invoking
pip install -r requirements.txt
. It is unnecessary and the documentation has been updated to show the changes.Closes #486