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

Monkey patch refactoring #3919

Merged
merged 2 commits into from
Aug 3, 2021
Merged

Monkey patch refactoring #3919

merged 2 commits into from
Aug 3, 2021

Conversation

mwutzke
Copy link
Contributor

@mwutzke mwutzke commented Aug 1, 2021

No description provided.

Doing the monkey patching as early as possible in the
application life-cycle, ensures that all uses of the
classes that are monkey patched, all have the same
implementation.
Now that the monkey patching has been relocated, a number
of the E402 overrides are no longer necessary.
@codecov
Copy link

codecov bot commented Aug 1, 2021

Codecov Report

Merging #3919 (d23e6a8) into master (6b2cd5e) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3919      +/-   ##
==========================================
- Coverage   92.47%   92.47%   -0.00%     
==========================================
  Files          47       47              
  Lines        8876     8873       -3     
==========================================
- Hits         8208     8205       -3     
  Misses        668      668              
Impacted Files Coverage Δ
faucet/faucet.py 74.87% <100.00%> (+0.27%) ⬆️
faucet/faucet_bgp.py 89.12% <100.00%> (-0.15%) ⬇️
faucet/faucet_dot1x.py 92.31% <100.00%> (-0.08%) ⬇️
faucet/faucet_event.py 60.00% <100.00%> (-0.47%) ⬇️

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 6b2cd5e...d23e6a8. Read the comment docs.

@anarkiwi anarkiwi merged commit 93cf890 into faucetsdn:master Aug 3, 2021
@mwutzke mwutzke deleted the monkey_patch branch August 3, 2021 02:55
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