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

ci: Support using git for dependencies in test-installation #222

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

Marenz
Copy link
Contributor

@Marenz Marenz commented Feb 7, 2024

Fix the CI test that fails when dependencies in pyproject.toml contain git urls.

I didn't have time to update the tests, feel free to take over.

@Marenz Marenz requested a review from a team as a code owner February 7, 2024 18:28
@github-actions github-actions bot added the part:template Affects the cookiecutter template files label Feb 7, 2024
@llucax llucax changed the title Fix test-installation dockerfile failing with git deps ci: Support using git for dependencies in test-installation Feb 8, 2024
@llucax llucax added part:ci Affects the GitHub workflow and other parts for running CI type:bug Something isn't working labels Feb 8, 2024
@llucax llucax added this to the v0.9.0 milestone Feb 8, 2024
@llucax
Copy link
Contributor

llucax commented Feb 8, 2024

Updating the tests should be just: UPDATE_GOLDEN=1 pytest tests/integration/test_cookiecutter_generation.py::test_golden.

Ideally this repo should get the fix.

I plan to dedicate some time to repo-config in a not so distant future, if you didn't take care of it until then I can take it over.

Copy link
Contributor

@llucax llucax left a comment

Choose a reason for hiding this comment

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

Marking as needing changes so it is more obvious in the PR list. Basically this needs:

  • Update this project with the fix
  • Fix tests
  • Split the cleaning of the docker image in its own commit
  • Improve the commit description a bit

llucax
llucax previously approved these changes Feb 19, 2024
@llucax
Copy link
Contributor

llucax commented Feb 19, 2024

OK, I took over this one and updated with the missing stuff. But now it needs an approval from someone else :)

@Marenz
Copy link
Contributor Author

Marenz commented Feb 19, 2024

Yeah, I can't approve this, being the author and all.

When git dependencies are used in the `pyproject.toml` file, the
`test-installation` CI job fails because git is not installed, so the
dependencies can't be retrieved. We install git in the container to fix
this.

Signed-off-by: Mathias L. Baumann <[email protected]>
Co-authored-by: Leandro Lucarella <[email protected]>
@Marenz Marenz disabled auto-merge February 20, 2024 10:46
@Marenz Marenz added this pull request to the merge queue Feb 20, 2024
@llucax llucax removed this pull request from the merge queue due to a manual request Feb 20, 2024
@llucax llucax added this pull request to the merge queue Feb 20, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit f479301 Feb 20, 2024
14 checks passed
@Marenz Marenz deleted the git-dep-inst-test branch February 20, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:ci Affects the GitHub workflow and other parts for running CI part:template Affects the cookiecutter template files type:bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants