Skip to content
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

extras: Pydantic Rut type regex is not compliant with JSON Schema #722

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

jtrobles-cdd
Copy link
Member

The regular expression used to validate RUTs in _RutPydanticAnnotation results in Pydantic generating an invalid JSON Schema because the regex uses named groups, which are not supported by the JavaScript regular expression syntax used by JSON Schema (and OpenAPI).

The regular expression used to validate RUTs in `_RutPydanticAnnotation`
results in Pydantic generating an invalid JSON Schema because the
regex uses named groups, which are not supported by the JavaScript
regular expression syntax used by JSON Schema (and OpenAPI).
@jtrobles-cdd jtrobles-cdd added bug Something isn't working component: rut labels Oct 25, 2024
@jtrobles-cdd jtrobles-cdd self-assigned this Oct 25, 2024
@jtrobles-cdd jtrobles-cdd requested a review from a team as a code owner October 25, 2024 02:17
Copy link

Copy link

codecov bot commented Oct 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.65%. Comparing base (7a433b2) to head (d885a3c).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #722      +/-   ##
===========================================
+ Coverage    87.63%   87.65%   +0.01%     
===========================================
  Files           38       38              
  Lines         3139     3142       +3     
  Branches       386      386              
===========================================
+ Hits          2751     2754       +3     
  Misses         246      246              
  Partials       142      142              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jtrobles-cdd jtrobles-cdd merged commit 02f52af into develop Oct 25, 2024
20 checks passed
@jtrobles-cdd jtrobles-cdd deleted the bugfix/pydantic-rut-json-schema-regex branch October 25, 2024 13:13
@svillegas-cdd svillegas-cdd mentioned this pull request Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component: rut
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants