Skip to content
Snippets Groups Projects

Clarify release process

Merged Luke Duncalfe requested to merge ld-clarify-release-process into master
All threads resolved!

What does this MR do and why?

This MR clarifies the release process. It came out of the explanation here !206 (comment 1375462361).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Reviewer roulette

    Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category and by a maintainer in all other categories.

    To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

    To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

    Once you've decided who will review this merge request, mention them as you normally would! Danger does not automatically notify them for you.

    Reviewer Maintainer
    Jennifer Li current availability (@jennli) (UTC-7, 19 hours behind @.luke) David Dieulivol current availability (@ddieulivol) (UTC+2, 10 hours behind @.luke)

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • Luke Duncalfe
  • Luke Duncalfe
  • Author Maintainer

    @godfat-gitlab What do you think about this change to the README.md to try to clarify the process for contributors who are new?

  • Luke Duncalfe requested review from @godfat-gitlab

    requested review from @godfat-gitlab

  • Lin Jen-Shin resolved all threads

    resolved all threads

  • Lin Jen-Shin approved this merge request

    approved this merge request

  • @.luke Looks great to me, thank you so much for improving the documentation and formalizing the process!

  • I am not sure why it doesn't allow me to merge and ask me to rebase:

    Screenshot_2023-05-04_at_01.26.17

    Guess I'll click it.

  • Lin Jen-Shin added 3 commits

    added 3 commits

    Compare with previous version

  • Lin Jen-Shin enabled automatic add to merge train when the pipeline for e0a5dc2d succeeds

    enabled automatic add to merge train when the pipeline for e0a5dc2d succeeds

  • Lin Jen-Shin started a merge train

    started a merge train

  • Lin Jen-Shin mentioned in commit d188ce67

    mentioned in commit d188ce67

  • merged

  • Please register or sign in to reply
    Loading