Skip to content

Remove committed uv.lock so the env resolves the latest ogcore#55

Open
SeaCelo wants to merge 1 commit into
EAPD-DRB:mainfrom
SeaCelo:chore/remove-uv-lock
Open

Remove committed uv.lock so the env resolves the latest ogcore#55
SeaCelo wants to merge 1 commit into
EAPD-DRB:mainfrom
SeaCelo:chore/remove-uv-lock

Conversation

@SeaCelo

@SeaCelo SeaCelo commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

uv.lock was committed automatically during the conda→uv migration, and it pins ogcore to the migration-era 0.15.13. So the documented setup does not pull the latest ogcore — on main, uv sync installs 0.15.13 (and even downgrades a newer install) even though 0.16.1 is released.

git checkout main
uv sync --extra dev
uv run python -c "import ogcore; print(ogcore.__version__)"

This stops committing uv.lock, so uv sync resolves the pyproject floors freshly → latest compatible (ogcore 0.16.1 today, new releases as they land). pyproject.toml keeps the ogcore>=… floor for compatibility (what package consumers already use); the lock is regenerated locally and git-ignored.

  • Removes uv.lock, adds it to .gitignore, updates the AGENTS.md convention.
  • Verified: uv sync now resolves ogcore 0.16.1; pytest -m 'not local' passes (21).
  • CI uses plain uv sync (no --frozen/--locked), so nothing breaks.

cc: @rickecon @jdebacker

(Same change as OG-ETH #51.)

@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.97%. Comparing base (3be67b8) to head (1c1bf15).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #55   +/-   ##
=======================================
  Coverage   85.97%   85.97%           
=======================================
  Files          13       13           
  Lines         649      649           
=======================================
  Hits          558      558           
  Misses         91       91           
Flag Coverage Δ
unittests 85.97% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@SeaCelo SeaCelo marked this pull request as ready for review June 12, 2026 13:55
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.

1 participant