Skip to content

Add acroitor as a backend maintainer

Alexandru Croitor requested to merge acroitor-backend-maintainer into master

Trainee maintainer issue: #8889 (closed)

Overview

I've been at GitLab since February 2019, working in the various forms of the groupproject management and part of the Groups and Projects Consolidation initiative in ~"group::workspace" as a backend engineer.

I've been in maintainer trainee program since August 2020.

I've contributed > 300 MRs, and have reviewed & approved almost 500 MRs as a backend reviewer and Database reviewer.

Examples of reviews

Some of the most recent MRs reviewed/approved

Reviews the code on my local machine to confirm behaviour

Contribute with patches as part of the review

Things to improve

There are always things to improve:

  • Learn more about different domains of the app (e.g. I am not so familiar with our CI/CD templating or infrastructure parts of the product)
  • Be more pro-acting on asking opinions of domain experts in certain MRs
  • I'm sure there is more 😄

@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 Donald Cook

Merge request reports