User Tools

Site Tools


contribution_guidelines

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
Last revision Both sides next revision
contribution_guidelines [2019/11/12 18:59]
be.ing
contribution_guidelines [2019/11/13 02:52]
tapir Enable 2FA on GitHub
Line 12: Line 12:
 Mixxx core team members are contributors who have write access to the [[https://​github.com/​mixxxdj/​|upstream mixxxdj repositories]] on GitHub, access to the Jenkins web interface for the build servers, and access to the private Zulip stream for the core team. Mixxx core team members are contributors who have write access to the [[https://​github.com/​mixxxdj/​|upstream mixxxdj repositories]] on GitHub, access to the Jenkins web interface for the build servers, and access to the private Zulip stream for the core team.
  
 +  * Enable [[https://​help.github.com/​en/​github/​authenticating-to-github/​securing-your-account-with-two-factor-authentication-2fa|two-factor authentication (2FA)]] for your GitHub account
   * //Never// force push to an upstream repository (mixxxdj). If you encounter an error from Git saying you would need to force push, stop what you are doing and discuss the situation on Zulip.   * //Never// force push to an upstream repository (mixxxdj). If you encounter an error from Git saying you would need to force push, stop what you are doing and discuss the situation on Zulip.
   * Only push directly to an upstream repository (mixxxdj) for trivial, uncontroversial changes like fixing a typo.   * Only push directly to an upstream repository (mixxxdj) for trivial, uncontroversial changes like fixing a typo.
Line 19: Line 20:
   * Check CI to ensure builds work and tests pass before merging. If CI timed out, either manually restart it or build the branch and run the tests locally before merging.   * Check CI to ensure builds work and tests pass before merging. If CI timed out, either manually restart it or build the branch and run the tests locally before merging.
   * Default to open; only post in the private Zulip stream for discussions that have a reason to be private. Most of the time, post to a public Zulip stream so anyone can participate in the discussion.   * Default to open; only post in the private Zulip stream for discussions that have a reason to be private. Most of the time, post to a public Zulip stream so anyone can participate in the discussion.
 +  * When Mixxx participates in Google Summer of Code, you may volunteer as a mentor if you like.
contribution_guidelines.txt ยท Last modified: 2019/11/13 02:53 by tapir