Skip to content

chore: refactor mr view

Kerri Miller requested to merge github/fork/zemzale/chore-refactor-mr-view into trunk

Created by: zemzale

Description Before working on improving mr view I refactored some parts. Mainly cleaned up the code to be more like issue_create.go where command options are declared before the command in a easy to understood struct, which makes the code a lot cleaner.

Also removed some unneded globals and added helper to IO and mrutils

  • chore(IO): Add helper for checking if output is TTY
  • chore(mrutils): Add custom option passing to MRFromArgs
  • ft(mr view): Refactor command structure

Related Issue

#322 (closed)

How Has This Been Tested?

Screenshots (if appropriate):

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)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)

Merge request reports