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

Add git_commit_detail directive. #21

Merged
merged 4 commits into from
Apr 13, 2015
Merged

Conversation

pomalley
Copy link
Contributor

This directive inserts information about the state of the repository
when sphinx is run to generate documentation.
The following options are available:
branch -- show the name of the branch
commit -- show the commit hash
sha_length -- number of (hex) characters to show for the commit
uncommitted -- show a warning if there are uncommited changes
untracked -- show a warning if there are untracked files

Update documentation to include git_commit_detail.
Add link to GitHub commit, with option to disable, and update the doc.

This directive inserts information about the state of the repository
when sphinx is run to generate documentation.
The following options are available:
branch -- show the name of the branch
commit -- show the commit hash
sha_length -- number of (hex) characters to show for the commit
uncommitted -- show a warning if there are uncommited changes
untracked -- show a warning if there are untracked files

Update documentation to include git_commit_detail.
Add link to GitHub commit, with option to disable, and update the doc.
@pomalley
Copy link
Contributor Author

I should add that if this doesn't fit with your vision of where sphinx-git is going, I'm fine just keeping my own fork.

@OddBloke
Copy link
Owner

I'd definitely love to merge this; let me review the PR.

First up: Could you look at adding some tests?

---------------------------

The ``git_commit_detail`` directive produces information about the current commit in the
repository against which the documentation is being built. The following options are availiable:
Copy link
Owner

Choose a reason for hiding this comment

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

s/availiable/available/

@OddBloke
Copy link
Owner

Have had a quick scan through and, broadly, this looks good. Just gimme some tests. :)

@pomalley
Copy link
Contributor Author

Yes of course. I got so excited I forgot about tests. :)

@pomalley
Copy link
Contributor Author

OK, if the tests look good to you I can squash the commits or you can take them as is.

@pomalley
Copy link
Contributor Author

Any chance this can get pulled in?

OddBloke added a commit that referenced this pull request Apr 13, 2015
Add git_commit_detail directive.
@OddBloke OddBloke merged commit 6b11bd2 into OddBloke:master Apr 13, 2015
@OddBloke
Copy link
Owner

Merged. Thanks @pomalley!

@pomalley
Copy link
Contributor Author

My pleasure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants