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

Don't recursively scan for pyproject.toml when calculating the cache key #260

Merged
merged 1 commit into from
May 2, 2024

Conversation

llucax
Copy link
Contributor

@llucax llucax commented May 2, 2024

We don't really need to scan for pyproject.toml files recursively, since we only have one in the root of the repository. This should make the cache key calculation more efficient and less error prone, as when using qemu, there are some files that are not accessible and the hash calculation fails.

We don't really need to scan for `pyproject.toml` files recursively,
since we only have one in the root of the repository. This should make
the cache key calculation more efficient and less error prone, as when
using qemu, there are some files that are not accessible and the hash
calculation fails.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax llucax requested a review from a team as a code owner May 2, 2024 09:47
@github-actions github-actions bot added the part:template Affects the cookiecutter template files label May 2, 2024
@llucax llucax added this to the v0.10.0 milestone May 2, 2024
@llucax llucax self-assigned this May 2, 2024
@llucax llucax enabled auto-merge May 2, 2024 09:51
@llucax
Copy link
Contributor Author

llucax commented May 2, 2024

Not sure why this says it still needs an approval from someone with write access, as @daniel-zullo-frequenz has write access 🤔

@llucax llucax disabled auto-merge May 2, 2024 11:00
@llucax llucax added this pull request to the merge queue May 2, 2024
@llucax llucax added the type:bug Something isn't working label May 2, 2024
@llucax
Copy link
Contributor Author

llucax commented May 2, 2024

I re-enabled auto-merge and it worked now 🤷

Merged via the queue into frequenz-floss:v0.x.x with commit b49a543 May 2, 2024
15 checks passed
@llucax llucax deleted the fix-pip-cache branch May 2, 2024 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants