-
-
Notifications
You must be signed in to change notification settings - Fork 222
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
Limit commit count from previous commit. #2026
Comments
@ankitghosh, can you please provide the complete log output from running the CLI? If possible, please run the CLI with the
As I already mentioned in the comment I left on #2006 when I closed the PR, this problem requires a different solution from what you suggest in #2006. Likely, we will need to introduce a separate option to limit the number of commits we consider for creating a new release. |
Hey i will try to generate and paste it here but in short it hangs after calling this Hmm agreed we could have a new flag, considering initial-depth is used for new release. But i think we should also have a default number which would prevent this from happening for users who does not provide a flag. |
Thank you in advance!
I understand your perspective here, but creating a default number is a breaking change, and so we will not be able to add a default number. For example, consider what would happen if we set the default number of commits for a release to 20, as you suggested in #2006. Let's say someone wants to create a new release, but they have 30 commits since the last release. Currently, this would work fine – the new release would be created with all 30 commits. However, with #2006, only the 20 most recent commits are included, even though the user probably would have intended for all 30 to be included. Adding a new flag should solve your problem, and we can also consider adding an error message for this command so that we inform users who encounter the same issue you did that they can use the flag to work around the problem. |
Yup fair point 👍 a new flag makes sense with error message in place. |
Steps to Reproduce
Recently our build started failing for uploading commit history to a build. So why it failed because we merged bunch of commits which were huge in number. So we were getting 413 error from
get_commits_from_git
function.Expected Result
Ideally commit should have been truncated with initial-depth value which we were passing.
Actual Result
It failed with 413 error.
Logs
I raise a PR as well for the fix #2006. Basically I am adding default_count in
get_commits_from_git
functionThe text was updated successfully, but these errors were encountered: