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

feat/python module composition #395

Merged
merged 3 commits into from
Oct 1, 2024

Conversation

blast-hardcheese
Copy link
Contributor

@blast-hardcheese blast-hardcheese commented Sep 18, 2024

Why

It's currently very challenging to iterate on python modules, as well as testing replacing pyright-extended with pyright+ruff. In support of that, let's just describe these as different modules.

These modules are not intended to be used directly, instead they will be toggled on/off via feature flag during env building in pid1.

What changed

  • Adding python-base-*, ruff, and pyright modules

Test plan

Compose these modules together and test the workspace. Ensure package installs are detected by pyright/ruff.

Rollout

  • This is fully backward and forward compatible

@blast-hardcheese blast-hardcheese force-pushed the dstewart/feat/python-module-composition branch 3 times, most recently from a096fea to 0f5e5d3 Compare September 20, 2024 22:55
@blast-hardcheese blast-hardcheese marked this pull request as ready for review September 20, 2024 22:55
@blast-hardcheese blast-hardcheese requested a review from a team as a code owner September 20, 2024 22:55
@blast-hardcheese blast-hardcheese requested review from vlinkz and removed request for a team September 20, 2024 22:55
@blast-hardcheese blast-hardcheese force-pushed the dstewart/feat/python-module-composition branch from 0f5e5d3 to 0cad6fc Compare October 1, 2024 19:59
@blast-hardcheese blast-hardcheese requested a review from a team as a code owner October 1, 2024 19:59
@blast-hardcheese blast-hardcheese requested review from airportyh and removed request for a team October 1, 2024 19:59
@blast-hardcheese blast-hardcheese merged commit 8538871 into main Oct 1, 2024
1 check passed
@blast-hardcheese blast-hardcheese deleted the dstewart/feat/python-module-composition branch October 1, 2024 20:06
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