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

Remove tests from pyproject #941

Merged
merged 2 commits into from
Jan 3, 2024
Merged

Conversation

Ninebell
Copy link

@Ninebell Ninebell commented Jan 2, 2024

when install neurokit2 by pip, tests folder in neurokit repository are installed in python env. they can conflict with other local tests directory.

This PR removed tests folder from setup.py by editing MANIFEST.in

Copy link

welcome bot commented Jan 2, 2024

Thanks for opening this pull request! We'll make sure it's perfect before merging 🤗 force
Make sure to read the contributing guide. Also, if you think that your contribution is worthy of it, you can consider adding yourself to the Contributors list (feel free to ask us if you have any doubts).

@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3d004b4) 54.86% compared to head (2c410f4) 54.61%.
Report is 17 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #941      +/-   ##
==========================================
- Coverage   54.86%   54.61%   -0.25%     
==========================================
  Files         303      304       +1     
  Lines       14255    14333      +78     
==========================================
+ Hits         7821     7828       +7     
- Misses       6434     6505      +71     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

MANIFEST.in Outdated
recursive-exclude * __pycache__
recursive-exclude * *.py[co]

recursive-include docs *.rst conf.py *.jpg *.png *.gif

prune tests
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can't we do recursive-exclude instead of prune?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, I applied it

@DominiqueMakowski
Copy link
Member

thanks!

@DominiqueMakowski DominiqueMakowski merged commit cdbefba into neuropsychology:dev Jan 3, 2024
8 checks passed
Copy link

welcome bot commented Jan 3, 2024

landing
Congrats on merging your first pull request! 🎉🍾 We're looking forward to your next one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants