Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Issue 11: Update contribution guidelines #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emilyJLin95
Copy link
Contributor

@emilyJLin95 emilyJLin95 commented Apr 14, 2022

Created a template for more consistent instruction format
Updated the readme to reflect a more structured process for box creation
DO NOT MERGE until box docs have been updated and truffle boxes have been cleaned up
Addresses: #11

README.md Show resolved Hide resolved
@leeftk
Copy link

leeftk commented May 2, 2022

LGTM!!!! Merge it!

Copy link
Member

@kevinbluer kevinbluer left a comment

Choose a reason for hiding this comment

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

One minor point re the bullet, otherwise looks awesome @emilyJLin95!

README.md Show resolved Hide resolved
- [REPLACE OR ADD REQUIREMENTS AND HOW TO DOWNLOAD]
- [Node.js](https://nodejs.org/) 10.x or later
- [NPM](https://docs.npmjs.com/cli/) version 5.2 or later
-
Copy link
Member

Choose a reason for hiding this comment

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

Look like this might be a spurious bullet point

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I left it there so that they can add more requirements, but I can remove it

Created a template for more consistent instruction format
Updated the readme to reflect a more structured process for box creation
DO NOT MERGE until box docs have been updated and truffle boxes have been cleaned up
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.

4 participants