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

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