Skip to content

Backport: fix stale remote URL for Python 3.8 (v0.16.2)#300

Merged
hasaant merged 2 commits into
wayfair-incubator:mainfrom
hasaant:backport/srvlc-2064-stale-remote-url
Apr 6, 2026
Merged

Backport: fix stale remote URL for Python 3.8 (v0.16.2)#300
hasaant merged 2 commits into
wayfair-incubator:mainfrom
hasaant:backport/srvlc-2064-stale-remote-url

Conversation

@hasaant

@hasaant hasaant commented Mar 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Backport of #298 for consumers still on Python 3.8 (e.g. dev-plats-acceleration-service).

pygitops 0.17.0+ requires Python 3.10+, so v0.18.0 can't be installed by services still running 3.8. This cherry-picks the same URL sync fix onto the v0.16.x line and bumps the version to 0.16.2.

The fix is identical: if the cached origin URL doesn't match the caller-provided repo_url, call set_url() before fetching.

Changes

Test plan

PH: SRVLC-2064

…16.2)

Cherry-pick of 1b93722 onto the v0.16.x branch for Python 3.8
compatibility. The acceleration service cannot use pygitops 0.17.0+
(requires Python 3.10+), so this backports the URL sync fix to
a release that supports Python >=3.7.

Fixes SRVLC-2064

Made-with: Cursor
@hasaant hasaant requested a review from jashparekh April 5, 2026 23:35
@hasaant hasaant enabled auto-merge (squash) April 6, 2026 17:52
Resolve conflicts:
- setup.cfg: accept deletion (project uses pyproject.toml on main)
- tests/test_operations.py: drop duplicate tests already present from wayfair-incubator#298

Made-with: Cursor
@hasaant hasaant merged commit 7cccd9f into wayfair-incubator:main Apr 6, 2026
13 checks passed
@codecov

codecov Bot commented Apr 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (623e1e7) to head (65e8548).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #300   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         7    +1     
  Lines          173       176    +3     
  Branches        20        16    -4     
=========================================
+ Hits           173       176    +3     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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