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

Add ghi lock and ghi unlock commands #313

Merged
merged 1 commit into from Jul 3, 2016

Conversation

davidcelis
Copy link
Contributor

@davidcelis davidcelis commented Jun 22, 2016

A while back, I added support for locking and unlocking issues via the API. This underwent a lengthy preview period that, as of today, is over.

With the preview media type no longer required in the Accept header, I thought I'd go ahead and quickly add the commands ghi lock and ghi unlock. These are both fairly straightforward commands, though I did keep the -m option that's also present on ghi close to provide a comment before locking (or unlocking) the given issue.

Signed-off-by: David Celis me@davidcel.is

A while back, I [added support for locking and unlocking issues via the
API](https://developer.github.com/changes/2016-02-11-issue-locking-api/).
This underwent a lengthy preview period that, as of today, [is
over](https://developer.github.com/changes/2016-06-22-issue-locking-api-is-now-official/).

With the preview media type no longer required in the `Accept` header, I
thought I'd go ahead and quickly add the commands `ghi lock` and `ghi
unlock`. These are both fairly straightforward commands, though I did
keep the `-m` option that's also present on `ghi close` to provide a
comment before locking (or unlocking) the given issue.

Signed-off-by: David Celis <me@davidcel.is>
@davidcelis
Copy link
Contributor Author

davidcelis commented Jun 22, 2016

Not totally sure what's up with the unrelated changes to ./ghi. Maybe some recent PR snuck in where the author forgot to run rake build? ¯_(ツ)_/¯

@shubhamshuklaer
Copy link
Collaborator

@davidcelis @AlexChesters
the unrelated change is from #301

@stephencelis stephencelis merged commit b321f7f into stephencelis:master Jul 3, 2016
@davidcelis davidcelis deleted the locking-and-unlocking branch July 5, 2016 17:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants