| 5 Apr 2025 |
| @tinybronca:sibnsk.net left the room. | 15:39:55 |
| 6 Apr 2025 |
| Lucy changed their profile picture. | 13:00:08 |
continous | So, I'm gonna be honest here, I don't quite understand why I was told to open an issue for tags to be added upstream for decora-wifi, and they just submitted an update that adds them. Are we hoping to pull the source from github? | 13:50:05 |
@hexa:lossy.network | yes, we are | 13:50:24 |
continous | Alright, I'll get to updating. Should be one commit then, yeah? | 13:50:46 |
@hexa:lossy.network | yup | 13:50:52 |
@hexa:lossy.network | tags are great, because we can autodetect new ones | 13:51:04 |
continous | I figured now that I'm kind of looking everything over. | 13:51:16 |
@hexa:lossy.network | * tags are great, because we can autodetect new ones and create automated updates | 13:51:38 |
continous | Honestly, I only packaged this so I can use Decora products on HASS, and wanted to contribute. Everyone has been so kind and helpful. 10/10 would contribute again. | 13:51:50 |
@hexa:lossy.network | so your package says | 13:52:17 |
@hexa:lossy.network | # No tests in Pypi source
doCheck = false;
| 13:52:23 |
@hexa:lossy.network | and we very much like to run the tests, since python generally just breaks at runtime | 13:52:40 |
@hexa:lossy.network | * # No tests in Pypi source
doCheck = false;
| 13:52:44 |
@hexa:lossy.network | hm, looks like it does not have an actual test suite | 13:53:37 |
@hexa:lossy.network | I guess you can just call python cli-test.py in `checkPhase | 13:53:59 |
@hexa:lossy.network | checkPhase = ''
runHook preCheck
python cli-test.py
runHook postCheck
'';
| 13:54:21 |
@hexa:lossy.network | https://github.com/tlyakhov/python-decora_wifi/blob/master/cli-test.py | 13:54:24 |
K900 | Pretty sure that "test" is more of a test tool | 13:55:12 |
K900 | Not an actual unit test | 13:55:15 |
@hexa:lossy.network | oh right | 13:56:43 |
@hexa:lossy.network | 🤷♂️ | 13:56:49 |
Geoffrey Frogeye | It seems that calendar-enabled Home Assistant installations on 24.11 now fail building with the message ical.tzif.timezoneinfo.TimezoneInfoError: Unable to load tzdata module: America/Coyhaique (https://hydra.nixos.org/build/293916363). Something with tzdata being more recent for python3Packages.tzdata. | 17:34:30 |
Geoffrey Frogeye | I came up with a fix (https://github.com/NixOS/nixpkgs/pull/396600), however didn't realize it would be a mass rebuild. Is it really true that rebasing to the right branch (from release-24.11 to staging-24.11) then force pushing, and then changing the GitHub target branch (if I understand https://github.com/NixOS/nixpkgs/blob/fcc5f57fb1ff58a54598ee686f6ff4ae251107fd/CONTRIBUTING.md#rebasing-between-branches-ie-from-master-to-staging correctly) will NOT send a notification to three thousands people? I'd rather look stupid asking now here than look stupid in many people's inbox later 😅. | 17:37:34 |
Geoffrey Frogeye | * It seems that calendar-enabled Home Assistant installations on 24.11 now fail building with the message ical.tzif.timezoneinfo.TimezoneInfoError: Unable to load tzdata module: America/Coyhaique (https://hydra.nixos.org/build/293916363). Something with tzdata being too recent for python3Packages.tzdata. | 17:38:52 |
| nikongen joined the room. | 17:41:20 |
@hexa:lossy.network | backporting tzdata is a bit much 😊 | 17:47:41 |
@hexa:lossy.network | what is the "calendar-enabled" home assistant? | 17:49:27 |
@hexa:lossy.network | oh, something broke the ical packaging on 24.11? meh | 17:50:12 |
Geoffrey Frogeye | Anything installation with local_calendar, local_todo or google, maybe more | 17:52:20 |