Bugfix: arraysize on Error shall be int, had wrong type #128
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We might consider some type of consolidation to define identical concepts in one place - for example many items need a datatype definition. But, comparing to Error to Argument, for example, one difference is that the Error item is optionally named but Arguments must be named.
Installation of the local package for development testing has changed, because calling setup.py directly is now deprecated (see multiple sources in python community for more details) Signed-off-by: Gunnar Andersson <gunnar.andersson@mercedes-benz.com>
gunnar-mb
force-pushed
the
bugfix_arraysize
branch
from
7572b66
to
cd70f0d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Author: Gunnar Andersson <gunnar.andersson@mercedes-benz.com>, MBition GmbH.
Bugfix: Arraysize on Error had wrong type
This was an obvious typo, noticed only now. Maybe some kind of redefinition would put this in a single place instead (DRY) but there are a few differences between concepts that might make that inconvenient. Worth considering later - this is more of a quick fix.
The program was tested solely for our own use cases, which might differ from yours.
The submission is provided under the main project license (LICENSE file in root of project).
Provider Information