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 pylint check for missing docstrings #106

Merged
merged 6 commits into from
Nov 19, 2021

Conversation

Michael-T-McCann
Copy link
Contributor

@Michael-T-McCann Michael-T-McCann commented Nov 17, 2021

The check is called "lint-optional" and can easily be extended to check more formatting.

closes #100

@codecov
Copy link

codecov bot commented Nov 17, 2021

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.44%. Comparing base (7fe0e6f) to head (8703ed1).
Report is 719 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   90.39%   90.44%   +0.05%     
==========================================
  Files          42       42              
  Lines        3027     3022       -5     
==========================================
- Hits         2736     2733       -3     
+ Misses        291      289       -2     
Flag Coverage Δ
unittests 90.44% <100.00%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@Michael-T-McCann Michael-T-McCann marked this pull request as ready for review November 18, 2021 22:13
@bwohlberg bwohlberg added the documentation Improvements or additions to documentation label Nov 19, 2021
@Michael-T-McCann Michael-T-McCann merged commit ca77fcd into main Nov 19, 2021
@Michael-T-McCann Michael-T-McCann deleted the mike/missing_docstring_100 branch November 19, 2021 14:18
@bwohlberg bwohlberg added the developer Developer environment: issues related to CI, git, etc. label Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer Developer environment: issues related to CI, git, etc. documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub workflow lint tests should be extended
2 participants