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.

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

Dependabot updating template dependencies #502

Open
lucasrcezimbra opened this issue Dec 26, 2024 · 3 comments · May be fixed by #506
Open

Dependabot updating template dependencies #502

lucasrcezimbra opened this issue Dec 26, 2024 · 3 comments · May be fixed by #506
Labels

Comments

@lucasrcezimbra
Copy link
Owner

Context

Dependabot is currently only updating the projects specified in the root pyproject.toml. To ensure all project dependencies are consistently updated, we need to extend this functionality to include dependencies located in the {{cookiecutter.project_slug}}/pyproject.toml. This will require creating a new workflow that runs in every PR and copies dependencies from the root pyproject.toml to {{cookiecutter.project_slug}}/pyproject.toml and runs poetry lock to lock the dependencies.

Acceptance Criteria

  • Add all dependencies from the root {{cookiecutter.project_slug}}/pyproject.toml to pyproject.toml in a separated group.
  • Implement a new workflow that:
    • Copies dependencies from the root pyproject.toml to {{cookiecutter.project_slug}}/pyproject.toml.
    • Executes poetry lock in the subdirectory to update the poetry.lock file.
    • It should run in every PR.
  • Verify that dependabot updates trigger for dependencies in both the root and {{cookiecutter.project_slug}}/pyproject.toml.

Out of Scope

  • Changes to dependency management tools other than Poetry.
  • Modifications to any non-Python dependency files in the repository.
Copy link

OpenHands started fixing the issue! You can monitor the progress here.

1 similar comment
@lucasrcezimbra
Copy link
Owner Author

OpenHands started fixing the issue! You can monitor the progress here.

@lucasrcezimbra
Copy link
Owner Author

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-502' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant