Skip to content

IAMMETER: support import/export#29791

Draft
CiNcH83 wants to merge 3 commits into
evcc-io:masterfrom
CiNcH83:iammeter_importexport
Draft

IAMMETER: support import/export#29791
CiNcH83 wants to merge 3 commits into
evcc-io:masterfrom
CiNcH83:iammeter_importexport

Conversation

@CiNcH83
Copy link
Copy Markdown
Contributor

@CiNcH83 CiNcH83 commented May 9, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've left some high level feedback:

  • Renaming energy to import may break existing configurations that rely on the previous key name; consider keeping energy as a backward-compatible alias or clearly handling/guarding this transition in the consuming code.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Renaming `energy` to `import` may break existing configurations that rely on the previous key name; consider keeping `energy` as a backward-compatible alias or clearly handling/guarding this transition in the consuming code.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@andig
Copy link
Copy Markdown
Member

andig commented May 9, 2026

Spannend. Warum failed das nicht im Test?

@andig andig added the devices Specific device support label May 9, 2026
@andig andig marked this pull request as draft May 10, 2026 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devices Specific device support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants