-
Notifications
You must be signed in to change notification settings - Fork 54
Development Coordination Process
C Custodio edited this page Jan 16, 2015
·
17 revisions
This is a proposed workflow for fixing issues and adding features to SEED.
- Use Github Issue Tracker to report the bug or request a new feature. Include as much detail as possible and screencaps, if it helps.
- Assign Issues to Developers
- Make a branch off of the "master" from the main code base or fork the SEED code to a new repository.
Difference betwen branches and forks - Make a branch in individual dev environment
- Change code and push changes to branch
- Test on individual development environment
- For large or long-running changes, periodically merge master down to branch and re-test.
- Issue pull request to SEED repository
- Develop acceptance tests
- Start review process to accept the branch into the main code 3. Unit tests (included in code) if possible (some issues will be easier to do than others) 3. UI testing + Manual testing, walk through the GUI + Selenium (screen capture feature) - Firefox only - + Jasmine 3. Visual code review + Minimum of one person, can bring in another person if needed 3. Documentation 3. Make sure changes are well documented + In code + PyDocs for Readthedocs
- Comments on pull request
- Review process concludes
- Accept pull-request
- Merge code into "master" branch in seed repository
Pull-requests are confirmed by the repository owner and should be tested first in the developer version of SEED before being merged with the official public version.
https://guides.github.com/activities/forking/#making-a-pull-request
SEED is brought to you by: U.S. DOE & Berkeley Lab–LBNL & National Renewable Energy Laboratory
- Information for Contributors
- Documentation guide
- Deployment
- Development
- Development Process
- General