-
Notifications
You must be signed in to change notification settings - Fork 73
Update onboarding documents, pending feedback #953
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -45,9 +45,22 @@ Post Acceptance | |
|
|
||
| Assuming the new developer has been accepted, the nominator: | ||
|
|
||
| * Makes sure the new developer agrees to the code of conduct and knows that discussions on dev channels should be considered private and may include work in progress and thus should not be shared. | ||
| * Makes sure the new developer gets access to the infrastructure (github, slack, mesa-dev, testhub). | ||
| * Acts as a mentor to the new developer, helping them to get used to the system and the way things are done. This includes making commits, merging PR's, and general development tasks. | ||
| * Makes sure the new developer agrees to the code of conduct and knows that discussions on developer channels should be considered private and may include work in progress that should not be shared. | ||
| * Makes sure the new developer gets access to the relevant infrastructure. This includes adding them to the MESAHub GitHub organization with appropriate permissions, adding them to Slack and any relevant private channels, and arranging access to mesa-dev and testhub. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Something to ponder on: to what level are we given access to the various aspects of the infrastructure. E.g. GitHub has different levels of access to a repository and organisation. |
||
| * Points the new developer to the existing documentation for development workflows. This includes contributing code through branches, making commits, opening pull requests, using the testing infrastructure, and identifying the points of contact for different parts of the infrastructure. | ||
| * Acts as the primary mentor for the new developer during onboarding, helping them to get used to the system and the way things are done. The nominator should make time to answer questions and help the new developer become comfortable with day to day development tasks. | ||
| * Helps the new developer learn the expected GitHub workflow, including when and how to create branches, make commits, open pull requests, respond to review, and, where appropriate, merge pull requests. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is the same as the third point |
||
| * Remains involved in reviewing and guiding the new developer's contributions for an initial onboarding period of at least six months, so that the new developer receives the attention and mentorship needed to become a routine contributor to MESA. | ||
| * Helps connect the new developer with other existing developers whose expertise or responsibilities are particularly relevant to the new developer's interests and planned areas of contribution. | ||
| * If the new developer expects to work on backward incompatible changes, encourages them to raise those plans with the broader developer team before substantial development begins. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would consider this more appropriate for the contribution guidelines, which would be already referenced in point 3. |
||
| * As the new developer becomes established, discusses with them whether they would like to take on additional project maintenance responsibilities. Such responsibilities are optional and are not a required part of onboarding. | ||
|
|
||
| The relevant onboarding contacts are: | ||
|
|
||
| * Mentor and primary onboarding contact: the nominator | ||
| * GitHub maintainer and contact for GitHub access: Matteo Cantiello | ||
| * Slack maintainer and contact for Slack access: Rich Townsend | ||
| * testhub maintainer and contact for testhub access: Bill Wolf | ||
|
|
||
| Infrastructure Access for Collaborators | ||
| --------------------------------------- | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed during the meeting, have a back-up mentor committed to assist if the nominator leaves the dev team during the on-boarding period.