-
Notifications
You must be signed in to change notification settings - Fork 0
Development Guidelines
Here we'll try to keep a list of development standards to be used in this project. This will be increasingly important for materials visible to students, but also for general organizational purposes.
The brainstorm repo is exactly what it sounds like: a place to test out our ideas and see what works. Here, I think we shouldn't worry too much about pull requests, code review, etc. and just let each member push freely to the repo.
However, when it comes time to begin committing to rescu_textbook, I think we should be a bit more careful. Before pushing material to the main text, we should make sure that everyone has the opportunity to review said material. Pull requests are a great way to do this. That way, everyone gets to see the change set before it gets merged. So a sample workflow might look like this:
- Fork
rescu/rescu_textbooktouser_foo_bar/rescu_textbook - Clone
user_foo_bar/rescu_textbook - Add an upstream to
rescu/rescu_textbookin your local repo - Got a new feature/chapter/section/whatever to add? Create a new branch!
- Do some work on the branch
- Create a pull request
- People comment
- You fix stuff
- More comments
- Repeat ad nauseum
- Once there is a consensus, your code gets merged!
How much do we want to utilize branches? dev? master? releases? versions?
Plots, coding, equations, etc.
For our sake and to alleviate confusion on the part of the students.
Not sure how necessary this will be. May become more apparent as amount of material grows.