Skip to content

Clarify contributing docs#2398

Merged
JelleZijlstra merged 1 commit into
mainfrom
JelleZijlstra-patch-1
Jul 24, 2021
Merged

Clarify contributing docs#2398
JelleZijlstra merged 1 commit into
mainfrom
JelleZijlstra-patch-1

Conversation

@JelleZijlstra
Copy link
Copy Markdown
Collaborator

"as configurable as gofmt" means little to people who haven't used gofmt.

"as configurable as gofmt" means little to people who haven't used gofmt.
@JelleZijlstra JelleZijlstra added the ci: skip news Pull requests that don't need a changelog entry. label Jul 24, 2021
@JelleZijlstra JelleZijlstra requested a review from ichard26 July 24, 2021 22:54
Copy link
Copy Markdown
Collaborator

@ichard26 ichard26 left a comment

Choose a reason for hiding this comment

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

Much better! Thank you! Hopefully this sets the stage of what Black is all about (even as we're trying to figure out the consistency bit right now) a bit better.

@ichard26 ichard26 added the T: documentation Improvements to the docs (e.g. new topic, correction, etc) label Jul 24, 2021
@JelleZijlstra JelleZijlstra merged commit 6665677 into main Jul 24, 2021
@ichard26 ichard26 deleted the JelleZijlstra-patch-1 branch July 24, 2021 23:00
ichard26 pushed a commit that referenced this pull request Aug 8, 2021
"as configurable as gofmt" means little to people who haven't used gofmt.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: skip news Pull requests that don't need a changelog entry. T: documentation Improvements to the docs (e.g. new topic, correction, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants