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

Unlock discussion #401

Merged
merged 3 commits into from
Dec 23, 2020
Merged

Unlock discussion #401

merged 3 commits into from
Dec 23, 2020

Conversation

maxice8
Copy link
Collaborator

@maxice8 maxice8 commented Dec 23, 2020

Description

Adds --unlock-discussion flags to mr update and issue update

Related Issue

Resolves #400

How Has This Been Tested?
Tested on gitlab.alpinelinux.org

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)

commands/mr/update/mr_update.go Outdated Show resolved Hide resolved
acts as the opposite of --lock-discussion
acts as the opposite of --lock-discussion
@maxice8
Copy link
Collaborator Author

maxice8 commented Dec 23, 2020

Moved them, also took the liberty to move the --public, --confidential check too

@profclems profclems merged commit d118ca7 into profclems:trunk Dec 23, 2020
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.

Allow 'issue' and 'mr' update to unlock discussion
2 participants