Skip to content

Switch to UV#228

Merged
jbonofre merged 5 commits into
apache:mainfrom
MonkeyCanCode:switch_to_uv_groupped
Jul 21, 2026
Merged

Switch to UV#228
jbonofre merged 5 commits into
apache:mainfrom
MonkeyCanCode:switch_to_uv_groupped

Conversation

@MonkeyCanCode

Copy link
Copy Markdown
Contributor

Summary

As per ML, we want to proceed with more standard layout for python project by using uv. Here are the changes made in this PR:

  1. Applied PEP 639 to python based model and converters
  2. Converted snowflake and honeydew converter to modern pyproject layout and modularized
  3. Remove un-necessary settings in pyprojects such as pythonpath = ["src"]
  4. Sorted the sections in pyprojects alphabetically
  5. Chcked-in uv.lock per project

Related Issues

Checklist

Specification

  • Spec changes are included in core-spec/ and follow the existing structure
  • Spec changes have been discussed on the mailing list or in a linked issue
  • Breaking changes to the spec are clearly called out in the summary

Ontology

  • Ontology changes in ontology/ are consistent with spec changes
  • New or modified terms are defined and documented

Converters

  • Converter logic in converters/ is updated to reflect spec or ontology changes
  • New converters include tests under the converter's test directory

Validation

  • Validation rules in validation/ are updated if the spec changed
  • New validation cases are covered by tests

Documentation

  • docs/ is updated to reflect any user-facing changes
  • New features or behaviors are documented with examples where appropriate
  • CONTRIBUTING.md is updated if the contribution process changed

Examples

  • examples/ are added or updated for any new spec constructs or converter support

Tests

  • All existing tests pass (pytest / CI green)
  • New functionality is covered by tests

Compliance

  • ASF license headers are present on all new source files
  • No third-party dependencies are added without PMC/IPMC approval

@QMalcolm QMalcolm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for doing this work! After we take care of the merge conflicts, we should be good to proceed 🙂

@MonkeyCanCode

Copy link
Copy Markdown
Contributor Author

Thank you for doing this work! After we take care of the merge conflicts, we should be good to proceed 🙂

Thanks for the review. Conflict had been resolved.

@MonkeyCanCode
MonkeyCanCode requested a review from QMalcolm July 20, 2026 21:30
@khush-bhatia
khush-bhatia self-requested a review July 21, 2026 05:01
khush-bhatia
khush-bhatia previously approved these changes Jul 21, 2026
@jbonofre
jbonofre self-requested a review July 21, 2026 05:40

@jbonofre jbonofre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It seems this PR removed the ASF header in omni and orionbelt converters. It's a requirement to have the ASF header, so we should keep it.

Comment thread converters/omni/pyproject.toml
Comment thread converters/orionbelt/pyproject.toml
@MonkeyCanCode

Copy link
Copy Markdown
Contributor Author

Thanks for the review @QMalcolm @khush-bhatia @jbonofre . I had addressed the last reported issue with missing ASF header. The missing ASF header had been restored now.

@jbonofre
jbonofre merged commit 8a73a38 into apache:main Jul 21, 2026
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.

4 participants