Skip to content

Refactor / small improvements#122

Merged
g-braeunlich merged 11 commits into
mainfrom
refactor
Jun 24, 2026
Merged

Refactor / small improvements#122
g-braeunlich merged 11 commits into
mainfrom
refactor

Conversation

@g-braeunlich

@g-braeunlich g-braeunlich commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

As a preparation for #112 some refactors and improvements:

  • improve typing a bit
  • suppress stacktrace if ctrl-c is pressed
  • ci: tests (except influxdb)
  • ci: migrate to post node 20 actions

@g-braeunlich g-braeunlich requested a review from aferk June 19, 2026 14:00
@g-braeunlich g-braeunlich force-pushed the refactor branch 4 times, most recently from 5fbb6ac to 0632549 Compare June 19, 2026 14:39
@MoritzFS

Copy link
Copy Markdown
Contributor

Deployed in the lab. Tested:

  • running scans
  • Setting params
  • Interfacing via scripting client

Works all nominally.

Comment thread src/icon/config/v1.py Outdated
Comment thread src/icon/config/config.py
Comment thread .github/workflows/test.yaml Outdated
@g-braeunlich g-braeunlich merged commit da2ddb0 into main Jun 24, 2026
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.

4 participants