这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@josegonzalez
Copy link
Member

Closes #1435

@josegonzalez
Copy link
Member Author

I think we can start here and iterate as we see what works?

@Flink
Copy link
Contributor

Flink commented Sep 9, 2015

From https://github.com/docker/docker/blob/master/CONTRIBUTING.md#quick-contribution-tips-and-guidelines there’s also a little section on git and I think we should incite good practices when writing commits like using rebase, squash commits, etc. (starting with us ;))

CONTRIBUTING.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

branche?

Copy link
Contributor

Choose a reason for hiding this comment

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

that’s french for branch 😁

@michaelshobbs
Copy link
Member

👍 on @Flink's comment

Closes #1435

[ci skip]
@josegonzalez
Copy link
Member Author

Updated the Making Changes section to take @Flink's comment into account. Merging :)

josegonzalez added a commit that referenced this pull request Sep 15, 2015
@josegonzalez josegonzalez merged commit 2ba9809 into master Sep 15, 2015
@josegonzalez josegonzalez deleted the 1435-josegonzalez branch September 15, 2015 03:44
@Flink
Copy link
Contributor

Flink commented Sep 15, 2015

Nice 👍

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.

4 participants