8000 README: Add explicit section on contributing. by pfalcon · Pull Request #3353 · micropython/micropython · GitHub
[go: up one dir, main page]

Skip to content

README: Add explicit section on contributing. #3353

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

Closed
wants to merge 1 commit into from

Conversation

pfalcon
Copy link
Contributor
@pfalcon pfalcon commented Oct 7, 2017

To increase visibility of Contributors' Guidelines and Code Conventions
docs.

To increase visibility of Contributors' Guidelines and Code Conventions
docs.
@pfalcon pfalcon requested a review from dpgeorge October 7, 2017 11:29
@pfalcon
Copy link
Contributor Author
pfalcon commented Oct 7, 2017

It seems that a lot of people miss to click that yellow-background link when submitting a pull request, so let's make contributing docs more visible.

Copy link
Member
@dpgeorge dpgeorge left a comment

Choose a reason for hiding this comment

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

I don't think it will help that much, but doesn't hurt.

productive, please be sure to follow
[Contributors' Guidelines](https://github.com/micropython/micropython/wiki/ContributorGuidelines)
and [Code Conventions](https://github.com/micropython/micropython/blob/master/CODECONVENTIONS.md)
8000 docs. Note that MicroPython is licenced under the MIT license, and all
Copy link
Member

Choose a reason for hiding this comment

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

Needs a few articles: "... follow the [Contributor's Guidelines] and the [Code Conventions]."

Also doesn't need the word "docs" because "Guidelines" and "Conventions" are already the noun.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@pfalcon
Copy link
Contributor Author
pfalcon commented Oct 15, 2017

I don't think it will help that much, but doesn't hurt.

Yeah, makes sense to do something, and browsing thru github projects, many have such a short section in their READMEs, so maybe they find that useful.

@pfalcon
Copy link
Contributor Author
pfalcon commented Oct 15, 2017

Merged.

@pfalcon pfalcon closed this Oct 15, 2017
@pfalcon pfalcon deleted the readme-contrib branch October 15, 2017 07:17
tannewt added a commit to tannewt/circuitpython that referenced this pull request Sep 30, 2020
Also fix two bugs:
* Crash when resetting board.SPI when it hasn't been inited.
* Reading back the output value is always false. Fixes micropython#3353
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.

2 participants
0