Skip to content

Conversation

@andig
Copy link
Member

@andig andig commented Aug 9, 2025

TODO

  • stable-sort commands and properties
  • order docs by tag

@andig andig added the enhancement New feature or request label Aug 9, 2025
Copy link
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 @andig - I've reviewed your changes and they look great!

Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments

### Comment 1
<location> `server/mcp/openapi.md:381` </location>
<code_context>
+
+## setGlobalSmartCostLimit
+
+Conveniance method to set smart charging cost limit for all loadpoints at once. Value is applied to each individual loadpoint.
+
+**Tags:** loadpoints
</code_context>

<issue_to_address>
Typo: 'Conveniance' should be 'Convenience'.

Update the description to use the correct spelling: 'Convenience method'.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
+## setGlobalSmartCostLimit
+
+Conveniance method to set smart charging cost limit for all loadpoints at once. Value is applied to each individual loadpoint.
+
+**Tags:** loadpoints
=======
+## setGlobalSmartCostLimit
+
+Convenience method to set smart charging cost limit for all loadpoints at once. Value is applied to each individual loadpoint.
+
+**Tags:** loadpoints
>>>>>>> REPLACE

</suggested_fix>

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 andig enabled auto-merge (squash) August 10, 2025 09:33
@andig andig merged commit 40453bb into master Aug 10, 2025
6 checks passed
@andig andig deleted the feat/mcp-naming branch August 10, 2025 09:42
andig added a commit that referenced this pull request Aug 11, 2025
thecem pushed a commit to thecem/evcc that referenced this pull request Sep 11, 2025
StarF666 pushed a commit to StarF666/evcc that referenced this pull request Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants