Skip to content

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Mar 2, 2023

Because why not.

Because why not.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 2, 2023

In https://github.com/thesofproject/sof/actions/runs/4318983392/jobs/7537920586 checkpatch is clueless and wrong

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Makes sense to add this as well, but I'd still add a bit of background to the git commit message.

uses: ./.github/workflows/pull-request.yml
run-zephyr-builds:
uses: ./.github/workflows/zephyr.yml
sparse-zephyr:
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd argue some better rationale could be added to the git commit that "why not".

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fair enough, I should probably have said something better like "there is no reason to exclude sparse from daily tests" :-)

Thanks for not blocking this.

@lgirdwood lgirdwood merged commit 98b4625 into thesofproject:main Mar 3, 2023
@aborisovich
Copy link
Contributor

Because that's why:
image

Please hotfix.

@marc-hb marc-hb deleted the sparse-mtl-daily branch March 5, 2023 21:53
@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 5, 2023

Thanks for spotting this, tentative fix submitted in:

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.

4 participants