Skip to content
This repository has been archived by the owner on Dec 7, 2020. It is now read-only.

Contributing section #95

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Contributing section #95

wants to merge 3 commits into from

Conversation

legodude17
Copy link

Adds a contributing section and updates the build section.

@legodude17
Copy link
Author

Oops, this includes my commits from #94. What is your take on this @getify?

@getify
Copy link
Owner

getify commented Nov 8, 2017

I am sorry I never circled back to provide feedback here.

What occurs to me, glancing at this, is that perhaps the best division of concerns should be that the main repo's README should not include the actual information about building contrib, but rather should mention the contrib-build and link to that section under the contrib README.

The reason I feel this way is that while both of these are in the same repo on github, they are separate packages in npm, so I don't want to clutter up too much of the main package's README with info about another package that someone wouldn't necessarily have.

A mention and link would seem a nice convenience, though.

Thoughts?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants