Skip to content

Conversation

@kiukchung
Copy link
Contributor

  • Replace setup.py with full PEP 621 pyproject.toml configuration
  • Update all 10 GitHub workflows to use uv instead of pip
  • Add uv.lock for reproducible builds with CI check in lint workflow
  • Add .pre-commit-config.yaml with optional uv lock check hook
  • Update README.md, CONTRIBUTING.md, and quickstart.md with uv instructions
  • Update .claude/ rules and scripts for uv-based development
  • Delete redundant requirements files (requirements.txt, dev-requirements.txt, docs/requirements.txt)
  • Update scripts/push_nightly.sh to use uv build

Test plan:
CI

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 24, 2026
@meta-codesync
Copy link

meta-codesync bot commented Jan 24, 2026

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D91406916. (Because this pull request was imported automatically, there will not be any future comments.)

@kiukchung kiukchung force-pushed the kiuk-dev branch 2 times, most recently from 4788726 to c0ffd18 Compare January 25, 2026 00:01
@codecov-commenter
Copy link

codecov-commenter commented Jan 25, 2026

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.62%. Comparing base (cefc805) to head (86b15cf).

Files with missing lines Patch % Lines
torchx/schedulers/test/train.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1199      +/-   ##
==========================================
+ Coverage   92.21%   92.62%   +0.41%     
==========================================
  Files          85       84       -1     
  Lines        6730     6700      -30     
==========================================
  Hits         6206     6206              
+ Misses        524      494      -30     
Flag Coverage Δ
unittests 92.62% <0.00%> (+0.41%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kiukchung kiukchung force-pushed the kiuk-dev branch 4 times, most recently from 8e572ad to a1eed03 Compare January 26, 2026 05:05
- Replace setup.py with full PEP 621 pyproject.toml configuration
- Update all 10 GitHub workflows to use uv instead of pip
- Add uv.lock for reproducible builds with CI check in lint workflow
- Add .pre-commit-config.yaml with optional uv lock check hook
- Update README.md, CONTRIBUTING.md, and quickstart.md with uv instructions
- Update .claude/ rules and scripts for uv-based development
- Delete redundant requirements files (requirements.txt, dev-requirements.txt, docs/requirements.txt)
- Update scripts/push_nightly.sh to use uv build
@kiukchung kiukchung merged commit 0543142 into main Jan 26, 2026
26 of 30 checks passed
@kiukchung kiukchung deleted the kiuk-dev branch January 26, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants