Please review the following if you intend to submit an issue or pull request. Not following the guidelines below most likely means your submission will be rejected.
ATTENTION: While creating tickets is the only way to assign a task to be completed, it is equally important to not flood the development team with tickets that are not actually necessary ( might be a duplicate, already being worked on, not actually an issue, etc ). So before you create a ticket, please check that your issue has not already been reported.
To submit a ticket, visit the project's repo on Github, select the Issues tab in the right column, and click the green New issue button.
Please use our Ticket Template to make sure you are entering all the information we will need to complete the item you are about to create.
So you want to create your own maps, or change some existing ones. Awesome. Make sure to check out the ./create folder for details on how to do this.
All source code for this project is contained in the ./src folder. Do not make any changes to the javascript files in the ./dist folder, these are automatically generated by the grunt build terminal command. Any Pull Requests that were not generated using the following Grunt Terminal Commands will be automatically rejected in order to maintain structural integrity of this project.
- Node.js v0.10 ( For Application Development )
You can use the following build commands via terminal:
The following command will Concat JS files and generate minified files for Distribution.
This is the most common command you will want to use and is required to view any changes you made in a browser.
grunt buildThe following will:
- Increase the build's major number ( e.g. v 1.0.1 => v 2.0.0 )
- Build & Package Distribution Files
- Perform a git commit
grunt release-majorThe following will:
- Increase the build's minor number ( e.g. v 1.0.1 => v 1.1.0 )
- Build & Package Distribution Files
- Perform a git commit
grunt release-minorThe following will:
- Increase the build's patch number ( e.g. v 1.1.1 => v 1.1.2 )
- Build & Package Distribution Files
- Perform a git commit
grunt release-patchCode will not be merged into the master branch unless it has been submitted as a Pull Request.
All Pull Requests must be reviewed by one of your peers before it can be merged into master.
- You have read and understand our Coding Culture
- You are working off a git branch named after a corresponding Github ticket
- You have merged in the latest master branch and resolved all merge conflicts
- Commit and push your Github ticket branch using Git Best Practices
- Visit the project's repo on Github and click Compare & pull request
- Add a Title that briefly explains the PR
- Use the text from our Pull Request Template as the base of your PR comment
- Update the PR comment to complete as much information as you can provide
- Add a Label of Ready for Code Review
- Click Create pull request
- Visit the PR you are being asked to review
- Before starting your PR Review, remove the Label Ready for Review and add the Label In Code Review
- Review the Files changed tab and add any inline comments ( click line number ) you have feedback with
- Add any overall feedback as a new comment on the main Conversation tab
- Once code review is complete, check off the items in the PR Reviewer once you have completed them
- If the PR fails, remove the Label In Code Review and select a new label showing what action is needed
- If the PR Passes, remove all Labels and click Merge pull request
- If you are 100% certain you will never need that branch again, you can delete the branch after merging into master
- Contact the PR Author and let them know the results of your review
- Stay on top of any PR's you are reviewing until they are able to be successfully merged into master