Skip to content

[WIP] [Do not merge] Try out pytest-twisted restructure branch#124

Closed
meejah wants to merge 2 commits intotahoe-lafs:masterfrom
meejah:patch-1
Closed

[WIP] [Do not merge] Try out pytest-twisted restructure branch#124
meejah wants to merge 2 commits intotahoe-lafs:masterfrom
meejah:patch-1

Conversation

@meejah
Copy link
Collaborator

@meejah meejah commented May 14, 2020

#123

(trying to get CI to build it)

@codecov
Copy link

codecov bot commented May 14, 2020

Codecov Report

Merging #124 into master will decrease coverage by 0.75%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
- Coverage   87.32%   86.57%   -0.76%     
==========================================
  Files          18       18              
  Lines        2115     2115              
  Branches      278      278              
==========================================
- Hits         1847     1831      -16     
- Misses        199      216      +17     
+ Partials       69       68       -1     
Flag Coverage Δ
#integration 61.56% <ø> (ø)
Impacted Files Coverage Δ
src/magic_folder/watchdog/_watchdog_541.py 71.42% <0.00%> (-14.29%) ⬇️
src/magic_folder/watchdog/inotify.py 81.08% <0.00%> (-11.72%) ⬇️
src/magic_folder/magic_folder.py 90.78% <0.00%> (-0.24%) ⬇️

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 ba3a155...3d038bd. Read the comment docs.

@altendky
Copy link

thanks again, seems ok i guess

@meejah
Copy link
Collaborator Author

meejah commented May 14, 2020

yup, integration tests pass and that's what uses pytest :)

@meejah meejah closed this May 14, 2020
@meejah meejah deleted the patch-1 branch June 20, 2020 05:48
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