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.

Sign up for GitHub

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

Jump to bottom

Reorganize Dependencies and CI packaging #540

Merged
merged 3 commits into from
Dec 2, 2024
Merged

Reorganize Dependencies and CI packaging #540

merged 3 commits into from
Dec 2, 2024

Conversation

teocns
Copy link
Contributor

@teocns teocns commented Dec 2, 2024 •
edited
Loading

Fixes #539

Details

  • Created a new [dependency-groups] to follow PEP 735
  • Moved tach package from dev dependencies to a dedicated ci group
  • Updated GitHub action tach-check to use pip install ".[ci]" which installs the package with CI dependencies

Testing

  • CI workflow should continue to run successfully with the new dependency structure
  • Development environment setup remains unchanged

@teocns teocns marked this pull request as ready for review December 2, 2024 14:29
@teocns teocns changed the title Reorganize Dependencies and CI Setup Reorganize Dependencies and CI packaging Dec 2, 2024
@teocns teocns enabled auto-merge (squash) December 2, 2024 14:44
@teocns teocns merged commit d664f02 into main Dec 2, 2024
1 check passed
@teocns teocns deleted the ci-packaging branch December 2, 2024 17:07
This was referenced Dec 16, 2024
Merged
Merged
Merged
Merged
Merged
Closed
Closed
Closed
Closed
Merged
Closed
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@areibman areibman areibman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

tach "can't be installed because it doesn't have a source distribution or wheel for the current platform"
2 participants
@teocns @areibman