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 description and help for local development #304

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

HelgeCPH
Copy link
Contributor

Since I am forgetful and do not remember which things are checked or enforced in PyDriller, I extend the developer documentation in the README, add a file with requirements that are needed for local development, and add a Makefile that should help me and others remembering what has to be checked before sending a pull-request.

Since I am forgetful and do not remember which things are checked or enforced
in PyDriller, I extend the developer documentation in the README, add a file
with requirements that are needed for local development, and add a Makefile
that should help me and others remembering what has to be checked before
sending a pull-request.
Copy link

testpulseio bot commented Oct 14, 2024

TestPulse report

Test execution

🥳 Congrats, all your tests have passed!
See all builds of the PR: https://app.testpulse.io/oss/ishepard/pydriller/builds/pr/304

Files without coverage

🥳 All files in the PR are covered by at least 1 test!

Coverage information

No changes in coverage, nice! 🎉

Copy link

codecov bot commented Oct 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.27%. Comparing base (51510ab) to head (3875541).
Report is 16 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #304   +/-   ##
=======================================
  Coverage   97.27%   97.27%           
=======================================
  Files          17       17           
  Lines        1102     1102           
=======================================
  Hits         1072     1072           
  Misses         30       30           

@ishepard
Copy link
Owner

LGTM!
Thanks for doing this! Instructions are very clear!

@ishepard ishepard merged commit b44153b into ishepard:master Oct 14, 2024
19 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