Skip to content

Adding contributing.md first draft#48

Merged
graysonarts merged 4 commits into
tableau:developmentfrom
graysonarts:contributing-file
Jul 11, 2016
Merged

Adding contributing.md first draft#48
graysonarts merged 4 commits into
tableau:developmentfrom
graysonarts:contributing-file

Conversation

@graysonarts

Copy link
Copy Markdown
Contributor

Fixes #41
First draft of the contributing.md. Please feel free to comment on wording, improvements, etc.

@benlower

benlower commented Jul 7, 2016

Copy link
Copy Markdown
Contributor

Let's get @lbrendanl and @jdomingu to weigh in too.

I like this. Would be good to add something like "If you are considering adding a new feature, we suggest starting a conversation about it first to identify any potential design or scheduling issues before jumping in."

@lbrendanl

Copy link
Copy Markdown

Would it be worth just briefly mentioning that issues and feature requests are done through GH issues and all other fixes are done through PRs? I don't think it's uncommon for some to be unfamiliar with GitHub so it may be worth calling out.

@t8y8

t8y8 commented Jul 8, 2016

Copy link
Copy Markdown
Contributor

Looks good to me

@graysonarts

Copy link
Copy Markdown
Contributor Author

@benlower I just saw your second comment about starting a conversation. I'll add something and send a new update real quick.

@benlower

benlower commented Jul 8, 2016

Copy link
Copy Markdown
Contributor

nice

Comment thread contributing.md Outdated
creating a PR can be found in the [github documentation](https://help.github.com/articles/creating-a-pull-request/)

If the feature is complex or has multiple solutions that could be equally appropriate approaches, it would be helpful to file an issue to discuss the
design trade-offs of each solution before implementing, to allow us to collective arrive at the best solution, which most likely exists in the middle

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

to collectively

@graysonarts

Copy link
Copy Markdown
Contributor Author

yeah, I english good. Thanks @t8y8, fixed

@graysonarts

Copy link
Copy Markdown
Contributor Author

Anymore feedback on this one? Or is it good to merge?

@jdomingu

Copy link
Copy Markdown
Contributor

Ship it!

@benlower

Copy link
Copy Markdown
Contributor

LGTM

@graysonarts graysonarts merged commit d9f90f3 into tableau:development Jul 11, 2016
@graysonarts graysonarts deleted the contributing-file branch July 11, 2016 20:09
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.

5 participants