Skip to content

fix(mr): Readd ability to close multiple MRs

Kerri Miller requested to merge github/fork/zemzale/b-mr-close-panic into trunk

Created by: zemzale

Description Ability to remove multiple MRs was there but it had gone missing when implementing MRFromArgs.

Added a function for concurrently fetching multiple merge requests from branches that, falsback to the MRFromArgs if the argument count is lower then 1. This is because the MRFromArgs, contains some specific logic for handling cases where there are no arguments or user provided 0 in arguments.

Added this functionality to deal with more then one MR to delete and reopen. I opted to not implement this functionality to merge command since, if there merge multiple MRs one might create merge conflicts to others and block the merging.

Related Issue Fixes #301 (closed)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Merge request reports