-
Notifications
You must be signed in to change notification settings - Fork 256
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
fix: division by zero in language stats #2171
base: main
Are you sure you want to change the base?
Conversation
Changed Packages
|
Signed-off-by: Jos Ahrens <[email protected]>
361a556
to
aa8a8c8
Compare
@yoramshai any chance you could see whether these changes fix your issue? |
Co-authored-by: Vincenzo Scamporlino <[email protected]> Signed-off-by: Jos Ahrens <[email protected]>
@vinzscam will try and update |
@vinzscam It doesn't look like the issue has been completely solved after updating the above package. |
@yoramshai 'completely'? does that mean you are experiencing partially improved behaviour or that it still gives the same issue as in the bug report? |
I'll check back once I'm back at work so I have a proper dataset to work with (would be new years), but yeah - that can't be intended. It'd be interesting to know what the GitHub API's response is to that Dockerfile analysis but I imagine I'd be able to figure out a reproduction case once I sit down with the project proper. |
Should fix #2160 (have not tested this particular change as I did not have a matching data set to reproduce)
Signed-off-by
line in the message. (more info)