bcgov / cas-reporting

This is for the Clean Growth Digital Services team for work related to reporting.
Apache License 2.0
0 stars 0 forks source link

Research spike: Look in to Merge Queues/Automerge to assist PR velocity #251

Open joshgamache opened 1 month ago

joshgamache commented 1 month ago

A merge queue would allow PRs to be brought into the core branch (develop) following our linear git history (rebase-based), with less effort on part of developers. As more of us work in the same monorepo, the likelihood of needing to rebase after a PR is in increases, and sometimes it's needed multiple times. It may be worthwhile exploring options to keep our git history as we like it, while preventing PRs from requiring rebase-hell. This was one of the potential wins we were hoping for from Graphite, prior to other considerations (cost). Other options are likely available at no cost, especially considering the environmental/government/open-source angle.

As a developer I want low-friction PR merging after approval because rebasing and waiting on CI/flaky tests can take substantial time and this will help me to reduce downtime waiting for PRs and automate away 'the boring bits'.

Potential options:

Questions to answer:

  1. Will this actually improve PR turnaround and Developer experience?
  2. What is the effort to implement?
  3. Dev adoption effort?
  4. Maintencance/infrastructure considerations?
  5. Can it work with the Monorepo?
  6. Cost?