Skip to content

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Aug 28, 2023

When debugging different behaviour between the GitHub workflow run and a local run it is very useful to know exactly which dependency versions are installed.

To do this we also add the nox session as a matrix dimention for the nox job, so it is easier to print the generated nox venv for the different sessions. This has the nice effect that nox max and min tests will run in parallel.

@llucax llucax requested a review from a team as a code owner August 28, 2023 08:39
@llucax llucax self-assigned this Aug 28, 2023
@llucax llucax added type:tech-debt Improves the project without visible changes for users part:cookiecutter Affects the generation of projects using cookiecutter part:ci Affects the GitHub workflow and other parts for running CI labels Aug 28, 2023
@llucax llucax added this to the v0.6.0 milestone Aug 28, 2023
@daniel-zullo-frequenz
Copy link
Contributor

There is just a conflict with the release notes

When debugging different behaviour between the GitHub workflow run and
a local run it is very useful to know exactly which dependency versions
are installed.

To do this we also add the `nox` session as a matrix dimention for the
`nox` job, so it is easier to print the generated `nox` venv for the
different sessions. This has the nice effect that nox max and min tests
will run in parallel.

Signed-off-by: Leandro Lucarella <[email protected]>
@llucax
Copy link
Contributor Author

llucax commented Aug 28, 2023

Updated and enabled auto-merge.

@github-actions github-actions bot removed part:cookiecutter Affects the generation of projects using cookiecutter part:ci Affects the GitHub workflow and other parts for running CI labels Aug 28, 2023
@llucax llucax enabled auto-merge August 28, 2023 13:20
@llucax llucax added this pull request to the merge queue Aug 28, 2023
Merged via the queue into frequenz-floss:v0.x.x with commit bb2e3a7 Aug 28, 2023
@llucax llucax deleted the ci-debug branch August 28, 2023 14:41
@llucax
Copy link
Contributor Author

llucax commented Aug 28, 2023

Very nice, the matrix silly change cut the CI time in half! From 20m to 10m! ⚡ 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:tech-debt Improves the project without visible changes for users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants