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 functionality to calculate project_activity for git #95

Merged
merged 2 commits into from Aug 3, 2018

Conversation

aswanipranjal
Copy link
Contributor

This PR adds the functionality to calculate project_activity section of the report for the git data source using the project_activity function.
It generates the timeseries aggregations for Commits class (Authors class already generates a timeseries which is being added to the function directly) for a given time interval.

This PR also adds tests and data for the timeseries generated.

This code adds the functionality to calculate the project activity
section of the report for the git data source
- This code adds the tests for the timeseries metrics generated by
Commits class
- It also adds the data for the same
@coveralls
Copy link

Pull Request Test Coverage Report for Build 241

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.02%) to 59.954%

Files with Coverage Reduction New Missed Lines %
/home/travis/build/chaoss/grimoirelab-manuscripts/manuscripts2/metrics/git.py 5 84.38%
Totals Coverage Status
Change from base Build 235: -0.02%
Covered Lines: 1036
Relevant Lines: 1728

💛 - Coveralls

@aswanipranjal
Copy link
Contributor Author

@valeriocos can you please review this and the other two PRs too?

@valeriocos
Copy link
Member

The PR looks good to me, but I don't understand why the coverage has decreased. Do you know why? Have you tried to run coveralls (https://pypi.org/project/coveralls/) locally to see which lines are missing?

@aswanipranjal
Copy link
Contributor Author

I haven't tried Coveralls, but I am on it! Thanks!

@aswanipranjal
Copy link
Contributor Author

screen shot 2018-08-03 at 11 53 17 am
screen shot 2018-08-03 at 11 53 28 am
These are the lines that are unreachable at the moment because I changed the tests. Hence the coverage has decreased.

@valeriocos
Copy link
Member

Thank you, @aswanipranjal

@valeriocos
Copy link
Member

LGTM

@valeriocos valeriocos merged commit 86b7263 into chaoss:master Aug 3, 2018
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.

None yet

3 participants