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

[FLINK-3611] [docs] corrected link in CONTRIBUTING.md #1786

Closed
wants to merge 2 commits into from

Conversation

s1ck
Copy link
Contributor

@s1ck s1ck commented Mar 13, 2016

No description provided.

@chiwanpark
Copy link
Member

Hi @s1ck, thanks for opening pull request! I would like to change this link to "Contributing Code" (http://flink.apache.org/contribute-code.html) page rather than "Coding Guidelines". How about this?

@s1ck
Copy link
Contributor Author

s1ck commented Mar 14, 2016

Sounds good, but then I would change the text to:
A detailed explanation can be found in our [Contribute Code Guide](http://flink.apache.org/contribute-code.html) which also contains a list of coding guidelines that you should follow.

@chiwanpark
Copy link
Member

+1 for your suggestion @s1ck.

@chiwanpark
Copy link
Member

+1, Looks good to me.

@fhueske
Copy link
Contributor

fhueske commented Mar 22, 2016

Merging

fhueske pushed a commit to fhueske/flink that referenced this pull request Mar 22, 2016
@asfgit asfgit closed this in 77c867a Mar 22, 2016
fijolekProjects pushed a commit to fijolekProjects/flink that referenced this pull request May 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants