Skip to content

Modernize build system (port to uv) - #2343

Merged
badboy merged 5 commits into
mozilla:mainfrom
wlach:uv-port
Aug 31, 2026
Merged

badboy merged 5 commits into
mozilla:mainfrom
wlach:uv-port

Conversation

@wlach

@wlach wlach commented Aug 18, 2026 •

Copy link
Copy Markdown
Contributor

NOT YET READY FOR REVIEW

Make the glean dictionary a standard python project, porting to uv and python 3.11. Simplifies local development a fair bit.

Pull Request checklist

  • The pull request has a descriptive title (and a reference to an issue it
    fixes, if applicable)
  • All tests and linter checks are passing
  • The pull request is free of merge conflicts

Comment thread pyproject.toml
"PyYAML==6.0.1",
"google-cloud-bigquery==3.25.0",
"google-cloud-storage==2.16.0",
"GitPython==3.1.41",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

May want to reconsider the hard pins at some point (that part is taken care of by uv lock) but 🤷

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.

I think the google-cloud once where explicitly pinned due to some issues with other versions. But I can't remember details. If that still applies I'd like to keep them pinned in the pyproject file, but gitpython and pyyaml could definitely be unpinned here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'll leave it for now, probably best only to change if there's genuine value to doing so

Comment thread .circleci/config.yml
- python-3.9-packages-v1-{{ .Branch }}-{{ checksum "requirements.in" }}-
- python-3.9-packages-v1-{{ .Branch }}-
- python-3.9-packages-v1-main-
- python-3.9-packages-v1-

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@wlach
wlach marked this pull request as ready for review August 23, 2026 20:26
@wlach
wlach requested a review from badboy August 23, 2026 20:26
@badboy
badboy enabled auto-merge (rebase) August 31, 2026 09:03

@badboy badboy 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.

🎉

@badboy
badboy disabled auto-merge August 31, 2026 09:04
@badboy
badboy merged commit 7d0d027 into mozilla:main Aug 31, 2026
11 checks passed
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.

2 participants