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

Docker_image.rst: remove pip install instruction #489

Closed
wants to merge 3 commits into from

Conversation

raghavgr
Copy link

@raghavgr raghavgr commented Nov 4, 2017

Docker_image.rst: remove pip install instruction

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 #486

@newbazz
Copy link
Contributor

newbazz commented Nov 5, 2017

LGTM.

@jayvdb
Copy link
Member

jayvdb commented Nov 5, 2017

ack f37de42

@jayvdb
Copy link
Member

jayvdb commented Nov 5, 2017

@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
@gitmate-bot
Copy link

Hey! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost!

@gitmate-bot
Copy link

Automated rebase was successful!

@jayvdb
Copy link
Member

jayvdb commented Nov 5, 2017

unack

@jayvdb
Copy link
Member

jayvdb commented Nov 5, 2017

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.

@jayvdb
Copy link
Member

jayvdb commented Nov 5, 2017

unack 40329da

@jayvdb
Copy link
Member

jayvdb commented Nov 5, 2017

(and rebase your commit)

@raghavgr
Copy link
Author

raghavgr commented Nov 6, 2017

@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
@jayvdb
Copy link
Member

jayvdb commented Nov 6, 2017

three commits instead of one.

@raghavgr raghavgr closed this Nov 6, 2017
@raghavgr raghavgr deleted the patch-1 branch November 6, 2017 18:23
@newbazz
Copy link
Contributor

newbazz commented Nov 6, 2017

Why closed this PR? Btw this happened with me also :P

@vipuldcoder
Copy link

vipuldcoder commented Dec 12, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants