Skip to content

STY: Docstrings and variable names#1470

Merged
MartinThoma merged 2 commits intomainfrom
minor-sty
Dec 10, 2022
Merged

STY: Docstrings and variable names#1470
MartinThoma merged 2 commits intomainfrom
minor-sty

Conversation

@MartinThoma
Copy link
Copy Markdown
Member

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 4, 2022

Codecov Report

Base: 94.01% // Head: 93.73% // Decreases project coverage by -0.27% ⚠️

Coverage data is based on head (915ec8a) compared to base (22214e8).
Patch coverage: 93.33% of modified lines in pull request are covered.

❗ Current head 915ec8a differs from pull request most recent head 9407b34. Consider uploading reports for the commit 9407b34 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1470      +/-   ##
==========================================
- Coverage   94.01%   93.73%   -0.28%     
==========================================
  Files          30       28       -2     
  Lines        5443     5235     -208     
  Branches     1038      997      -41     
==========================================
- Hits         5117     4907     -210     
- Misses        197      199       +2     
  Partials      129      129              
Impacted Files Coverage Δ
PyPDF2/_page.py 92.22% <92.85%> (ø)
PyPDF2/_encryption.py 92.66% <100.00%> (ø)
PyPDF2/generic/_data_structures.py 95.11% <0.00%> (-0.51%) ⬇️
PyPDF2/filters.py 97.29% <0.00%> (-0.02%) ⬇️
PyPDF2/_utils.py
PyPDF2/__init__.py

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@MartinThoma MartinThoma merged commit 9821468 into main Dec 10, 2022
@MartinThoma MartinThoma deleted the minor-sty branch December 10, 2022 07:23
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.

1 participant