Skip to content

Conversation

MacGH23
Copy link
Contributor

@MacGH23 MacGH23 commented Mar 31, 2025

Update Standalone version to latest changes

@MacGH23
Copy link
Contributor Author

MacGH23 commented Mar 31, 2025

The
ubms_can
is also missing in the main version
bms/ubms_can.by

@@ -353,7 +353,7 @@ def check_bms_types(supported_bms_types, type) -> None:
elif port.startswith(("can", "vecan", "vcan")):
"""
Import CAN classes only if it's a CAN port; otherwise, the driver won't start due to missing Python modules.
This prevents issues when using the driver exclusively with a serial connection.
This prevents issues when using the driver exclusively with a serial connection
Copy link
Owner

Choose a reason for hiding this comment

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

Please add the dot, so that the file won't change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, hm, i didn't edit this one I thought

@mr-manuel
Copy link
Owner

Make sure all checks were successful.

@MacGH23
Copy link
Contributor Author

MacGH23 commented Mar 31, 2025

The Error of ubms_can.by is still present, but this file is missing in the master, too

@mr-manuel mr-manuel merged commit b203ef7 into mr-manuel:master Mar 31, 2025
1 of 2 checks passed
mr-manuel pushed a commit that referenced this pull request Mar 31, 2025
@mr-manuel
Copy link
Owner

@MacGH23 you mixed tab indentation with spaces.

@MacGH23
Copy link
Contributor Author

MacGH23 commented Mar 31, 2025

Hm, where ?
I use VSCode with Black Formatter.
Perhaps I have a setting wrong.

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.

2 participants