konstin / poc-monotrail

Proof Of Concept for python package management without virtualenvs
25 stars 4 forks source link

Add monotrail-utils crate #53

Closed cnpryer closed 1 year ago

cnpryer commented 1 year ago

UPDATED:

This PR moves RequirementsTxt to a new monotrail-utils crate.

Summary of changes:

  1. Add monotrail-utils crate
  2. Move RequirementsTxt over
  3. Use RequirementsTxt in poetry_integration

Tag https://github.com/mitsuhiko/rye/pull/265 Closes #56

konstin commented 1 year ago

Move the poetry integration usage into monotrail-utils

Could you use the monotrail_util RequirementsTxt in the poetry integration?

We also need updated CI workflows to publish monotrail-utils to crates.io, not necessarily now but before it blocks downstream crates from publishing to crates.io

cnpryer commented 1 year ago

Could you use the monotrail_util RequirementsTxt in the poetry integration?

I'll check it out.

cnpryer commented 1 year ago

I can look into making a PR for the CI after if that's alright with you.

I'm planning on getting https://github.com/mitsuhiko/rye/pull/265 ready for a review next. Then I can loop around to the CI.

cnpryer commented 1 year ago

I have the same 12 pass 1 failure (wheel::test::installed_paths_relative) on main for cargo test --workspace. I'll look again later.

UPDATE: #56

cnpryer commented 1 year ago

I can create an issue for this, but on main cargo test --release results with

failures:
    test_flipstring
    test_pipx_black_version
    test_poetry_config
    test_srcery

I'm on macos.

UPDATE: https://github.com/konstin/poc-monotrail/issues/55

konstin commented 1 year ago

Thanks!