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

gh_commit: Add get command **WIP** #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jcfr
Copy link
Collaborator

@jcfr jcfr commented Nov 8, 2017

Todo:

  • By default list all properties as:

    githubrelease commit REPO_NAME get SHA
    

    could output:

    sha 7921bb62e3b3240226acf66a57535219b9c3abac
    url https://api.github.com/repos/NeurodataWithoutBorders/pynwb/git/commits/7921bb62e3b3240226acf66a57535219b9c3abac
    html_url https://github.com/NeurodataWithoutBorders/pynwb/commit/7921bb62e3b3240226acf66a57535219b9c3abac
    author.name John Doe
    [...]
    
  • support getting a single property using

    githubrelease commit REPO_NAME get SHA author.name
    
  • support getting json output using

    githubrelease commit REPO_NAME get SHA --json
    

[ci skip]

Todo:

* By default list all properties as:

   githubrelease commit REPO_NAME get SHA

   could output:

   sha 7921bb62e3b3240226acf66a57535219b9c3abac
   url https://api.github.com/repos/NeurodataWithoutBorders/pynwb/git/commits/7921bb62e3b3240226acf66a57535219b9c3abac
   html_url NeurodataWithoutBorders/pynwb@7921bb6
   author.name John Doe
   [...]


* support getting a single property using

  githubrelease commit REPO_NAME get SHA author.name

* support getting json output using

  githubrelease commit REPO_NAME get SHA --json




[ci skip]
@jcfr jcfr added the wip label Apr 8, 2018
@jcfr jcfr force-pushed the master branch 4 times, most recently from deb6fef to c2ef301 Compare March 29, 2020 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant