fix: various custom operator conformance fixes#386
fix: various custom operator conformance fixes#386leakonvalinka wants to merge 3 commits intoopen-feature:mainfrom
Conversation
Signed-off-by: Lea Konvalinka <lea.konvalinka@dynatrace.com>
Signed-off-by: Lea Konvalinka <lea.konvalinka@dynatrace.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the test-harness subproject, introduces partial version support in SemVer parsing by padding with zeros, and changes the return value of string comparisons for non-string inputs from False to None. The review feedback identifies a bug in the version padding logic that breaks versions with suffixes, notes that existing tests need updates to reflect the new return types, and suggests maintaining backward compatibility for the renamed parse_version function.
| ends_with, | ||
| fractional, | ||
| parse_version, | ||
| normalize_version, |
There was a problem hiding this comment.
Renaming parse_version to normalize_version is a breaking change for consumers of this module. Given that this file is explicitly intended for backward compatibility (as noted in the header comment), it should continue to export parse_version as an alias.
| normalize_version, | |
| normalize_version, | |
| normalize_version as parse_version, |
There was a problem hiding this comment.
Not sure about this one, is this necessary? I changed it locally but the pre-commit hook reverted it :D
There was a problem hiding this comment.
i think it is is ok. this is an internal function, I don't think anyone relied on it. just to be a bit more carful let's add it as a change in the description.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #386 +/- ##
==========================================
- Coverage 96.14% 96.04% -0.10%
==========================================
Files 47 33 -14
Lines 1737 1139 -598
==========================================
- Hits 1670 1094 -576
+ Misses 67 45 -22 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Lea Konvalinka <lea.konvalinka@dynatrace.com>
leakonvalinka
force-pushed
the
fix/evaluator-inconsistencies
branch
from
4cbf9f0 to
ee5e7b6
Compare
| def test_v_prefix(self) -> None: | ||
| assert sem_ver({}, "v2.0.0", "=", "2.0.0") is True | ||
|
|
||
| def test_partial_version(self) -> None: |
There was a problem hiding this comment.
can you also add the v2.1 -. 2.1.0 test?
4 participants
parse_versiontonormalize_versionFixes: #379
Related: