Skip to content

fix(bart26g): correct spotoptim path dependency so Dependabot can resolve it#101

Merged
bartzbeielstein merged 2 commits into
mainfrom
copilot/fix-dependabot-failure
Jun 13, 2026
Merged

fix(bart26g): correct spotoptim path dependency so Dependabot can resolve it#101
bartzbeielstein merged 2 commits into
mainfrom
copilot/fix-dependabot-failure

Conversation

Copilot AI commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Dependabot was failing with path_dependencies_not_reachable on spotoptim in bart26g/pyproject.toml because the declared path ../../spotoptim resolves outside the cloned repository.

Changes

  • bart26g/pyproject.toml: Fix path source for spotoptim../../spotoptim.. (repo root)

    # Before
    spotoptim = { path = "../../spotoptim", editable = true }
    
    # After
    spotoptim = { path = "..", editable = true }

    The uv.lock had already been generated with editable = "../", confirming the pyproject.toml held a stale/wrong value.

  • bart26g/uv.lock: Regenerated to sync with the corrected path — bumps spotoptim 0.12.41.0.1 and drops now-unreferenced transitive packages (torch, tensorboard, statsmodels, and others).

…to ..

The path `../../spotoptim` pointed outside the repository, causing
Dependabot to fail with `path_dependencies_not_reachable`. The correct
relative path from bart26g/ to the repo root (where spotoptim lives)
is `..`. Also regenerates uv.lock to reflect the current spotoptim
version (1.0.1) and remove now-unused transitive packages.
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Dependabot fix(bart26g): correct spotoptim path dependency so Dependabot can resolve it Jun 13, 2026
Copilot AI requested a review from bartzbeielstein June 13, 2026 05:55

@bartzbeielstein bartzbeielstein left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@bartzbeielstein bartzbeielstein marked this pull request as ready for review June 13, 2026 07:19
@bartzbeielstein bartzbeielstein merged commit 4322792 into main Jun 13, 2026
5 checks passed
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