Skip to content

Smhi fix from old PR - #87

Merged
mosbth merged 11 commits into
mosbth:masterfrom
Challe-P:smhi-fix-frompr
Aug 23, 2026
Merged

Smhi fix from old PR#87
mosbth merged 11 commits into
mosbth:masterfrom
Challe-P:smhi-fix-frompr

Conversation

@Challe-P

Copy link
Copy Markdown
Contributor

Continues on the work of PR 86 (#86).
SMHIs API has been changed, breaking the weather feature of marvin. This PR solves the issue of marvin not being able to get the current temperature by using another API endpoint.
Tests and test data have been changed to match the new format.

kh31d4r and others added 9 commits August 8, 2025 11:39
Use a regex with a capture group instead of repetitive
if-statements. This makes it possible to add more budord without
changing the code, only strings.json.
On request from Efo.
If one incoming message is matching multiple actions, only reply
once. This will make the multi line response look like a single block,
as the subsequent output will follow right after the reply.
To automatically connect to the server and join a channel.
Restructured the repo to look more like a modern python
project.

This has multiple impacts across the repo:
 - README has been updated to reflect the new uv commands
 - Github actions has been updated to use uv instead of Make
 - Uv will now build a python package
 - The python package will be installed in the docker image, instead
   of just copying loose files
 - Static configuration files are now packaged inside the package
One file per action, in preparation of decoupling the actions and make
them plugins.
@mosbth
mosbth merged commit 12c1626 into mosbth:master Aug 23, 2026
4 of 5 checks passed
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.

3 participants