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

Closed
wants to merge 1 commit into from

Conversation

raghavgr
Copy link

@raghavgr raghavgr commented Nov 6, 2017

Docker_image.rst: remove pip install instruction

pip install is unnecessary in Docker image docs.

Closes #486

`pip install` is unnecessary in Docker Image docs.

Closes coala#486
@newbazz
Copy link
Contributor

newbazz commented Nov 6, 2017

In Docker Image write i instead of I, otherwise LGTM.

Copy link

@neelanshsahai neelanshsahai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ack 03185b6

@raghavgr
Copy link
Author

raghavgr commented Nov 7, 2017

@newbazz I have updated Docker Image to Docker image.

Copy link
Member

@Monal5031 Monal5031 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@raghav16179 You have sent a PR before assigning the issue, this is not how things are done.
Please read the newcomers guide first before starting to contribute to coala, it will help you in all your queries.

@Monal5031
Copy link
Member

unack 03185b6

@jayvdb
Copy link
Member

jayvdb commented Nov 9, 2017

@raghavgr , please do not close PRs and open new ones, like you have done with #489 and this one.
Also as @Monal5031 has pointed out, you are not assigned to the issue #486 ; someone else is assigned to it now, so I am closing this pull request.

Follow http://coala.io/newcomer carefully if you want to be part of the coala organisation.

@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.

8 participants