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

Request for open review #61

Closed
wants to merge 9 commits into from
Closed

Request for open review #61

wants to merge 9 commits into from

Conversation

nulltoken
Copy link
Member

  • Better handling of UTF-8 (credit goes to @xpaulbettsx)
  • Tags and Branches lazy loading
  • (Very) minor tweaks and fixes
  • Improved enumeration of commit

@davidfowl long ago you were after support for git log --all.

This should now be achievable with the following syntax

using (var repo = new Repository(yourRepoPath))
{
      var commits = repo.Commits.QueryBy(new Filter { Since = repo.Refs });
}

which is more or less in accordance with the git log man page which states

--all
    Pretend as if all the refs in [refs/] are listed on the command line as [<commit>].

@tclem
Copy link
Member

tclem commented Sep 13, 2011

👊 Looks good to me.

@nulltoken
Copy link
Member Author

@tclem thanks for the review.

I've added a small feature (allowing a TagAnnotation as a starting point when querying commits) and refactored some of the CommitFixture tests.

Merged and applied.

@nulltoken nulltoken closed this Sep 14, 2011
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.

3 participants