-
Notifications
You must be signed in to change notification settings - Fork 11
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
38 changed files
with
5,348 additions
and
3,908 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,49 +1,38 @@ | ||
|
||
## Contributing ## | ||
## Contributing | ||
|
||
Please read this doc before start to use this repo. We have a set of guidelines that should be followed. | ||
|
||
Feel free to propose changes to this document using a pull request. | ||
|
||
* Code patterns | ||
|
||
### Available Scripts ### | ||
|
||
Local Development enviroment | ||
``` | ||
npm start (component-name) | ||
``` | ||
Build for production | ||
``` | ||
npm run build | ||
``` | ||
Run unit testing | ||
``` | ||
npm test | ||
``` | ||
### Available Scripts | ||
|
||
See [README](./README.md#available-scripts). | ||
|
||
### Git Workflow ### | ||
### Git Workflow | ||
|
||
We use [GitFlow](http://nvie.com/posts/a-successful-git-branching-model/). | ||
|
||
### Branching | ||
### Branching | ||
|
||
- ```feature/feature-name``` to new features. | ||
- ```fix/bug-name``` to fix bugs. | ||
- ```improvement/improvement-name``` to improvement code. | ||
|
||
### Commit messages ### | ||
### Commit messages | ||
|
||
- Feel free to commit clear messages, in **english** to describe every single change in your branch. | ||
- We use [conventional commits](https://conventionalcommits.org/) as guidelines for merge commits. | ||
|
||
### Pull Requests ### | ||
### Pull Requests | ||
|
||
All Pull Request must be made to the `develop branch`. | ||
|
||
Before opening a Pull Request, verify if all tests ppassesassed, | ||
we have CI tools to run tests, if it fails [you will be notified](https://www.youtube.com/watch?v=mmLRTVYgEq4). | ||
|
||
|
||
### Releasing a new version ### | ||
### Releasing a new version | ||
|
||
Open a PR to the develop and master branchs. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.