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 full git history fetch to ci.yml #4

Merged
merged 1 commit into from
Apr 2, 2024
Merged

Add full git history fetch to ci.yml #4

merged 1 commit into from
Apr 2, 2024

Conversation

dariusptrs
Copy link
Member

@hofbi I forgot we need this so that git rev-list HEAD --count returns the correct commit number for \GitRevision. Thats why I had it initally in my previous PR.

@hofbi
Copy link
Member

hofbi commented Apr 2, 2024

Is there another way to get the number of commits? Did it work before?

@dariusptrs
Copy link
Member Author

We could replace the number with the commit SHA (optionally also again with a hyperlink to the commit) or maybe GitHub API offers an option to get the total number of commits.
Right now it always returns 1 as the commit number.

@hofbi
Copy link
Member

hofbi commented Apr 2, 2024

The current option causes "0 indicates all history for all branches and tags.". We only need the full histroy of the main/master branch. If that it easy to configure let's do it. Else we can also go for this since we don't expect a huge histroy with many branches.

@dariusptrs
Copy link
Member Author

I think its not that easily configurable. You could to something like git fetch --depth=1000000 which then is branch specific and will probably cause no problems. But it is more of a workaround...

@hofbi hofbi merged commit ad82392 into latex4ei:master Apr 2, 2024
2 checks passed
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