New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-40675: Implement schema version specification for RFC-954 #23
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #23 +/- ##
==========================================
+ Coverage 97.96% 98.12% +0.16%
==========================================
Files 6 6
Lines 393 427 +34
==========================================
+ Hits 385 419 +34
Misses 8 8
☔ View full report in Codecov by Sentry. |
.pre-commit-config.yaml
Outdated
@@ -7,8 +7,9 @@ repos: | |||
- "--unsafe" | |||
- id: end-of-file-fixer | |||
- id: trailing-whitespace | |||
exclude: "README.md" | |||
- repo: https://github.com/psf/black |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that black now has a specialist repo for pre-commit at https://github.com/psf/black-pre-commit-mirror
.pre-commit-config.yaml
Outdated
@@ -22,6 +23,6 @@ repos: | |||
- id: isort | |||
name: isort (python) | |||
- repo: https://github.com/PyCQA/flake8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use ruff in pre-commit?
651a535
to
719673f
Compare
README defines the structure for `version` keyword that can be either a simple string or a structure specifying version compatibility. Visitor classes are extended to handle version information, for now only the `SimpleVisitor` converts version info into a usable structure.
719673f
to
aa09a2a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks okay. All the manual model validation seems painful to me in a world where Pydantic exists. Please modernize type annotations.
README defines the structure for
version
keyword that can be either asimple string or a structure specifying version compatibility. Visitor
classes are extended to handle version information, for now only the
SimpleVisitor
converts version info into a usable structure.Plus some code cleanup and ruff checks for docstrings.