Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

mr: deprecate mr for command #900

Merged
merged 1 commit into from
Nov 4, 2021
Merged

mr: deprecate mr for command #900

merged 1 commit into from
Nov 4, 2021

Conversation

profclems
Copy link
Owner

The mr for command works just like create but accepts
issue ID as argument.

It creates a related merge request for the specified.
But for some reasons, it lacks some options available
when using mr create.

This change deprecates the mr for command and
adds additional options to mr create to perform
be able to create related merge request for issues

The `mr for` command works just like create but accepts
issue ID as argument.

It creates a related merge request for the specified.
But for some reasons, it lacks some options available
when using `mr create`.

This change deprecates the `mr for` command and
adds additional options to `mr create` to perform
be able to create related merge request for issues
@profclems profclems linked an issue Nov 4, 2021 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Nov 4, 2021

Codecov Report

Merging #900 (b2709a5) into trunk (4e9c673) will decrease coverage by 0.20%.
The diff coverage is 14.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            trunk     #900      +/-   ##
==========================================
- Coverage   59.42%   59.22%   -0.21%     
==========================================
  Files          95       95              
  Lines        7485     7516      +31     
==========================================
+ Hits         4448     4451       +3     
- Misses       2644     2670      +26     
- Partials      393      395       +2     
Impacted Files Coverage Δ
commands/mr/create/mr_create.go 31.52% <14.00%> (-1.72%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e9c673...b2709a5. Read the comment docs.

Copy link
Collaborator

@zemzale zemzale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@zemzale zemzale merged commit 13977b1 into trunk Nov 4, 2021
@zemzale zemzale deleted the deprecate-mr-for branch November 4, 2021 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mr for $x - unable to assign user "glab mr for" returns 400 - ref is empty Deprecate glab mr for command
2 participants