...
- Create an issue at https://openwsn.atlassian.net. Let's call it "FW-123"develop
- Assign it to yourself
- Click the "start progress" button
- Your issue is now marked "in progress". This ensures people know you're working on a fix.
- Assign it to yourself
- Fork the repository you want to modify under your username
- Navigate to the the repository you want to modify, on GitHub. Click the "fork" button
- This will create an exact copy of the entire repository, but under your name. That is, "https://github.com/openwsn-berkeley/openwsn-fw" is forked to "https://github.com/myusername/openwsn-fw".
- Optionally, rename your fork
openwsn-fw_FW-123
.
- Navigate to the the repository you want to modify, on GitHub. Click the "fork" button
- Implement a fix for the issue
- Commit and /push all your changes into that branchto your fork.
- Start your every commit message by the issues key, followed by a "
.
" , for example:FW-123. Fixed the packet format of RPL DIO.
This will ensure that your commit activity appears in the "CommitsDevelopment" tab section of your issue. - From time to time, merge the
develop
branch the main repository into your privatedevelop_FW-123
branchfork.
- When you're done
- Merge
develop
intodevelop_FW-123
Merge the main repository into your fork one last time, - From the GitHub interface, create a pull request by clicking the "create pull request" button
- In the issue tracker, click the "Resolve Issue".
Mark the - Your issue will now appear as "Fixed".
- resolved"
- This tells everybody you consider being done. Normally, you have nothing else to do.
- Merge
- The integration managers now merge your fork into the main code
- If people have comments on your fix, you can discuss directly through GitHub. This can be done line-by-line, making discussion and commenting extremely efficient.
- If you want to make further changes, no need to close the pull request, just continue making changes to your branch, your pull request will reflect those further changes automatically.
- Once the discussion is over and you have done final adjustments to your code, the integration manager merges your pull request is accepted and your
develop_FW-123
branch is merged into thedevelop
branch. - After the fix is tested, the integration manager marks the issue as "Closed".
- You can then into the main repository. On GitHub, your pull request now appears as "merged"
- The integration manager marks your issue as "closed".
- You can now delete your fork.
- If people have comments on your fix, you can discuss directly through GitHub. This can be done line-by-line, making discussion and commenting extremely efficient.