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
Impact Ratio as a new metric #226
Comments
@yegor256 should we count LOC only in |
@zaqbez39me yes, of course. There are no other files available in the dataset, when metric calculation starts. |
@yegor256 Should we split this task on several PRs? I mean, each subtask test and implementation should be in separate PRs, or all tests in first PR and all the code in second PR? |
@zaqbez39me definitely, would be better to do this in a number of PRs. I would suggest to start from tests. |
@yegor256 In IRLoC, IRC and IRCA we have repo and some file. Do repo and file paths should be provided separately to the script? If yes, should we check that the file is a part of a given repo? If no, should we change dir to parent of the given file and call there all the git commands needed? |
@zaqbez39me I think, we can find the "root" directory of the repo by looking for the |
Let's add a series of new metrics:
The text was updated successfully, but these errors were encountered: