Skip to content

Add Doug Stull as Backend maintainer (Gitlab)

Doug Stull requested to merge 5430-backend-trainee-maintainer-doug-stull into master

Overview

I have been a Senior Fullstack Engineer at GitLab for 16 months, as a member of groupexpansion, being a trainee maintainer for over a year. In that time I have these stats:

  • overall: 523 Merged MRs, 296 reviewed MRs that were merged.
  • gitlab-org/gitlab: 178 Merged MRs, 147 reviewed MRs that were merged.
  • gitlab-services/version-gitlab-com: 175 Merged MRs, 85 reviewed MRs that were merged.
  • gitlab-services/customers-gitlab-com: 42 Merged MRs, 51 reviewed MRs that were merged.
  • gitlab-org/license-gitlab-com: 42 Merged MRs, 5 reviewed MRs that were merged.
  • Maintainer-On: CustomersDot, Versions App(backend and frontend), LicenseDot, gitlab-styles, gitlab-experiment
  • Trainee Maintainer: GitLab(Database)
  • gitlab-org/gitlab-styles: 23 Merged MRs with an effort to help MR review cycle time by investing in standardization and automatic linting.

Examples of reviews

Things to improve

  • Domain knowledge.
  • Familiarity with graphql/grape, background jobs, caching.
  • Wording on feedback to be blameless and help lead to a solution(everyday improvement).
  • More focus on testing analysis.

Trainee maintainer issue: #5430 (closed) (lightly maintained as the length of time as a trainee made it hard to keep fresh)

@gitlab-org/maintainers/rails-backend please chime in below with your thoughts, and approve this MR if you agree.

Developer checklist

  • Before this MR is merged
    • Mention @gitlab-org/maintainers/rails-backend, if not done (this issue template should do this automatically)
    • Assign this issue to your manager
  • After this MR is merged
    • Request a maintainer from the #backend_maintainers Slack channel to add you as an Owner to gitlab-org/maintainers/rails-backend
    • Consider adding 'backend maintainer' to your Slack notification keywords

Manager checklist

Edited by Phil Calder

Merge request reports