An error occurred while loading the file. Please try again.
-
Lorenz Detterbeck authoredff0f29e1
After you've reviewed these contribution guidelines, you'll be all set to
contribute to this project.
Forked from an inaccessible project.
CONTRIBUTING.md 2.63 KiB
How to contribute
This project is due to a 'Softwaretechnik-Projekt' at THM in WiSe 17/18, from March 1st until March 16th. If you want to contribute, please contact our product owner.
Contact Information will be added when he gives his okay.
Horizon
With this project, a new frontend for ARSnova based on Angular will be build.
We want to build this frontend as lightweight as possible while granting a user friendly, envolving experience.
For more information, see our README.md.
Getting Started
- Make sure you have a THM Gitlab account.
- Submit an issue for every task if it does not already exist.
- Clearly describe the issue including steps to reproduce when it is a bug.
Making changes
- Create a topic branch from where you want to base your work.
- This should always be the master branch, unless something went terribly wrong.
- Use tags to describe your issue/branch/merge request.
- Always provide information on what you are working on.
- To quickly create a topic branch, go to your issue, expand
Create a merge request
and selectCreate branch
.
- Make commits of logical and atomic units.
- Check for unnecessary whitespaces with
git diff --check
before commiting. - Create tests for your changes, if possible (yes, it takes time, it is annoying, but also it is necassary)
- Run all tests to assure nothing else was accidentally broken.
Submitting Changes
- Push your changes to a topic branch in the repository.
- Check whether your topic branch is up to date with
master
. If not, please rebase your branch. - Submit a merge request to the repository.
- Provide information about what changed.
- Mark you merge request with
ready for testing
when you finished your work. If you haven't already, mark it withwork in progress
and add[WIP]:
to the merge request's title. - The team will then test your changes. When everything is as expected, your merge request will be marked as
ready for review
- The scrum master will then check your code for style and compatibility. If everything is okay, your changes will be merged.
- If something goes wrong - do not panic! There will be a change request with more information about what went wrong. You can fix these problems, your merge request will remain open.
Styleguides
Git Commit Messages
- Use the present tense (
Add feature
instead ofAdded feature
). - Use the imperative mood (
Move cursor to...
instead ofMoves cursor to...
). - Limit the first line to 72 characters or less.
- Reference issues and merge requests liberally after the first line.