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

In master/v2 can no longer git rev-parse HEAD^ to get prior commit #91

Closed
ajcann opened this issue Dec 4, 2019 · 5 comments · Fixed by #112
Closed

In master/v2 can no longer git rev-parse HEAD^ to get prior commit #91

ajcann opened this issue Dec 4, 2019 · 5 comments · Fixed by #112
Assignees
Labels
documentation Improvements or additions to documentation question Further information is requested

Comments

@ajcann
Copy link

ajcann commented Dec 4, 2019

Currently using packtracker which relies on being able to access the prior commit via git rev-parse HEAD^ reverting to v1.2 works.

@dijam
Copy link

dijam commented Dec 4, 2019

I have the same problem. For now, reverted from master to v1.

@ericsciple
Copy link
Contributor

Setting fetch-depth: 2 should work

@ericsciple ericsciple added the question Further information is requested label Dec 4, 2019
@ericsciple
Copy link
Contributor

note, the reason is because actions/checkout@v2 performance was optimized for the mainline scenario: fetch a single commit.

@ajcann
Copy link
Author

ajcann commented Dec 5, 2019

Makes sense - thanks for the context. Happy to close this unless you'd like to keep it open.

@ericsciple ericsciple self-assigned this Dec 5, 2019
@ericsciple
Copy link
Contributor

@ajcann thanks for confirmation. I might leave this open for a little bit longer. I might add this as an example scenario in the readme

@ericsciple ericsciple added the documentation Improvements or additions to documentation label Dec 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants