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.

Sign up for GitHub

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

Jump to bottom

[Question]nbgv get-commits #348

Closed
MarcoRossignoli opened this issue Jun 3, 2019 · 4 comments
Closed

[Question]nbgv get-commits #348

MarcoRossignoli opened this issue Jun 3, 2019 · 4 comments

Comments

@MarcoRossignoli
Copy link
Member

I'm testing usage of nbgv get-commits commands on coverlet repo https://github.com/tonerdo/coverlet/commits/master

I would like to know if it's expected that if I ask for last version done by only 1 commit it returns "no results"

D:\git\coverlet\src\coverlet.collector (master -> origin)
λ nbgv get-commits 1.0.25

D:\git\coverlet\src\coverlet.collector (master -> origin)
λ nbgv get-commits 1.0.24
e5998d8fbd07376dd8908e4829dda208d82d582d 1.0.24.39397 Merge pull request #451 from AArnott/versionPatch

I would expect 6209239d697b3e2fbd0ddfbe4e3d6e7e6cc6c212 from 1.0.25
Nightly for reference https://www.myget.org/feed/coverlet-dev/package/nuget/coverlet.collector/1.0.25-g6209239d69

I would like prepare a doc with "nightly usage" infos and the idea is document how get commit hash from version.

cc: @AArnott @tonerdo
@MarcoRossignoli
Copy link
Member Author

Never mind my fault...I was not synched with master.

@AArnott
Copy link
Collaborator

AArnott commented Jun 3, 2019

You'll need to be using the 3.0-beta for your repo.
And FYI, I found a bug in this area in the 3.0 version that I've yet to fix.

@AArnott
Copy link
Collaborator

AArnott commented Jun 3, 2019

#347 is the issue I found.

@MarcoRossignoli
Copy link
Member Author

I'm using

nbgv                                   2.3.167                     nbgv

and it works

D:\git\coverlet\src\coverlet.collector (master -> origin)
λ nbgv get-commits 1.0.25
6209239d697b3e2fbd0ddfbe4e3d6e7e6cc6c212 1.0.25.2402 Merge pull request #441 from AArnott/fixPackageOutputPath

AArnott pushed a commit that referenced this issue Feb 25, 2025
@renovate
Update dependency Microsoft.NET.Test.Sdk to 17.13.0 (#348)
a8d08d4 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@AArnott AArnott mentioned this issue Feb 25, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AArnott @MarcoRossignoli