Open hannahbaumann opened 2 weeks ago
Hello @hannahbaumann! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 91.47%. Comparing base (
76af603
) to head (a92d517
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
IMHO this should get a news entry since this change could cause a system to go from working to not working.
IMHO this should get a news entry since this change could cause a system to go from working to not working.
Agreed - this is more of a proof of concept at the moment, @RiesBen is taking over testing for this. We'll need to do a few PLB systems to convince ourselves that this works.
Checklist
news
entryDevelopers certificate of origin