Skip to content

refactor(issue/view): extract issuable/list package

Vitali Tatarintev requested to merge 1209-extract-issuable-list-package into main

Description

  • Extract issuable/list package
  • Combine tests from issue/list and incident/list into a single issuable/list test file.

The first part of the refactoring can be found here !1163 (merged)

In order to keep the commit history of the issue_list.go and issue_list_test.go files, the files were moved in the first commit and refactored in the second. Therefore, the commits should not be squashed in this MR.

Example of the kept history.

Related Issues

Resolves #1209 (closed)

How has this been tested?

  1. Run make build
  2. Run glab issue list
  3. Run glab issue list --issue-type=incident
  4. Run glab incident list

The last three commands should work as before

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)
  • Refactoring

Closes #1209 (closed)

Edited by Vitali Tatarintev

Merge request reports