Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gists #32

Closed
dotps1 opened this issue Jul 16, 2018 · 2 comments · Fixed by #172
Closed

Gists #32

dotps1 opened this issue Jul 16, 2018 · 2 comments · Fixed by #172
Labels
api completeness This is basic API functionality that hasn't been implemented yet. enhancement An issue or pull request introducing new functionality to the project. in progress Work on this issue is already underway. Please don't work start new work on it.

Comments

@dotps1
Copy link

dotps1 commented Jul 16, 2018

Just thought id throw this out there, but i have a pretty stable module out there that could be included in this for working with Gists: https://github.com/dotps1/PSGist

i haven't updated it in a while, but defiantly could put some work into it if it where included in this repo.

thanks!

@HowardWolosky HowardWolosky added api completeness This is basic API functionality that hasn't been implemented yet. up for grabs Anyone in the community is welcome to do this work help wanted Anyone in the community is welcome to do this work enhancement An issue or pull request introducing new functionality to the project. labels Nov 30, 2018
@raghav710
Copy link

@dotps1 was any work on this done?

@HowardWolosky
Copy link
Member

It might be harder to directly integrate in given the difference in authentication behavior.

@HowardWolosky HowardWolosky added in progress Work on this issue is already underway. Please don't work start new work on it. and removed help wanted Anyone in the community is welcome to do this work up for grabs Anyone in the community is welcome to do this work labels May 15, 2020
HowardWolosky added a commit that referenced this issue Jul 20, 2020
This completes the required work to support the full set of API's around gists.

It adds the following functions:
* `Get-GitHubGist`
* `Remove-GitHubGist`
* `Copy-GitHubGist` (aka. `Fork-GitHubGist`)
* `Add-GitHubGistStar`
* `Remove-GitHubGistStar`
* `Set-GitHubGistStar` (just a wrapper around `Add/Remove-GitHubGistStar`
* `Test-GitHubGistStar`
* `New-GitHubGist`
* `Set-GitHubGist`
   * `Rename-GitHubGistFile` (exposed helper function)
   * `Remove-GitHubGistFile` (exposed helper function)
   * `Set-GitHubGistFile` (exposed helper function, also known as `Add-GitHubGistFile`)
* `Get-GitHubGistComment`
* `Set-GitHubGistComment`
* `New-GitHubGistComment`
* `Remove-GitHubGistComment`

This also adds formatters for all newly introduced types: `GitHub.Gist`, `GitHub.GistCommit`, `GitHub.GistDetail`, and `GitHub.GistFork`.

Positional Binding has been set as `false`, and `Position` attributes added to the functions' mandatory parameters.

References:
[GitHub Gist](https://developer.github.com/v3/gists/)
[GitHub Gist Comments](https://developer.github.com/v3/gists/comments/)

Fixes #32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api completeness This is basic API functionality that hasn't been implemented yet. enhancement An issue or pull request introducing new functionality to the project. in progress Work on this issue is already underway. Please don't work start new work on it.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants