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

Port allmydata.util.log to Python 3 #761

Merged
merged 11 commits into from Aug 7, 2020
Merged

Port allmydata.util.log to Python 3 #761

merged 11 commits into from Aug 7, 2020

Conversation

itamarst
Copy link
Collaborator

@itamarst itamarst commented Aug 3, 2020

Copy link
Member

@exarkun exarkun left a comment

Choose a reason for hiding this comment

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

Thanks. Just a couple minor comments inline, otherwise looks good. Please address and merge.

src/allmydata/test/test_log.py Outdated Show resolved Hide resolved
src/allmydata/test/test_log.py Show resolved Hide resolved
@itamarst
Copy link
Collaborator Author

itamarst commented Aug 5, 2020

Looks like some legit test failures.

@itamarst itamarst closed this Aug 6, 2020
@itamarst itamarst reopened this Aug 6, 2020
@itamarst
Copy link
Collaborator Author

itamarst commented Aug 6, 2020

More test failures :(

@codecov-commenter
Copy link

codecov-commenter commented Aug 7, 2020

Codecov Report

Merging #761 into master will increase coverage by 0.01%.
The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #761      +/-   ##
==========================================
+ Coverage   91.21%   91.23%   +0.01%     
==========================================
  Files         156      155       -1     
  Lines       27235    27223      -12     
  Branches     3873     3871       -2     
==========================================
- Hits        24842    24836       -6     
+ Misses       1671     1665       -6     
  Partials      722      722              
Impacted Files Coverage Δ
src/allmydata/hashtree.py 99.42% <ø> (ø)
src/allmydata/util/_python3.py 66.66% <ø> (ø)
src/allmydata/util/log.py 98.07% <91.66%> (+0.40%) ⬆️
src/allmydata/stats.py 85.71% <0.00%> (-0.87%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 596cf09...fb9bf55. Read the comment docs.

@itamarst itamarst merged commit 5d70a78 into master Aug 7, 2020
@itamarst itamarst deleted the 3365.log-python-3 branch August 7, 2020 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants