PTST / O365-HomeAssistant

Office 365 integration for Home Assistant
MIT License
69 stars 13 forks source link

Fix for all day_event #82

Closed PTST closed 4 years ago

PTST commented 4 years ago

fixes #53

sourcery-ai[bot] commented 4 years ago

Sourcery Code Quality Report (beta)

❌  Merging this PR will decrease code quality in the affected files by 0.06 out of 10.

Quality metrics Before After Change
Complexity 1.97 2.11 0.14 🔴
Method Length 54.19 56.62 2.43 🔴
Quality 8.62 8.56 -0.06 🔴
Other metrics Before After Change
Lines 445 459 14
Changed files Quality Before Quality After Quality Change
custom_components/o365/calendar.py 8.68 8.60 -0.08 🔴
custom_components/o365/sensor.py 8.49 8.48 -0.01 🔴

Here are some functions in these files that still need a tune-up:

File Function Complexity Length Overall Recommendation
custom_components/o365/sensor.py O365QuerySensor.__init__ 28 308.71 2.90 Split out functionality
custom_components/o365/sensor.py O365InboxSensor.__init__ 12 140.90 5.79 Split out functionality
custom_components/o365/calendar.py setup_platform 8 159.87 5.89 Split out functionality
custom_components/o365/calendar.py O365CalendarData.async_update 5 154.83 6.25 Split out functionality

Please see our documentation here for details on how these metrics are calculated.

We are actively working on this report - lots more documentation and extra metrics to come!

Let us know what you think of it by mentioning @sourcery-ai in a comment.

PTST commented 4 years ago

Partial fix for #61