Skip to content

Update Ryan from trainee maintainer to maintainer in CustomersDot

Ragnar Hardarson requested to merge master-patch-cd78 into master

Manager Justification

  • The MRs reviewed by the candidate consistently make it through maintainer review without significant additionally required changes.
  • The MRs authored by the candidate consistently make it through reviewer and maintainer review without significant required changes.
  • Ryan has been a backend reviewer for CustomersDot for ~2.5 years
  • In that time Ryan has diligently kept code quality high with reviews and his own contributions
  • Ryan has reviewed ~70 MRs within the CustomersDot repository.
  • Ryan has authored ~102 MRs within the CustomersDot repository.

Before Merging (Manager Tasks)

  • Mention the maintainers from the given specialty and ask them to provide feedback to the manager directly.
  • Leave this merge request open for 1 week, to give the maintainers time to provide feedback.
  • Ensure we have at least 2 approvals from existing maintainers.

Once This MR is Merged

  1. Create an access request for maintainer access to gitlab-org/<project>.
  2. Join the #backend_maintainers Slack channel
  3. Ask the maintainers in your group to invite you to any maintainer-specific meeting if one exists.
  4. Let a maintainer add you to @gitlab-org/maintainers/rails-backend
  5. Announce it everywhere
  6. Keep reviewing, start merging 🤘 😎 🤘

Merge request reports