Skip to content

Add an extra style when no checks are present#656

Open
mchehab wants to merge 1 commit into
getpatchwork:mainfrom
mchehab:no_checks
Open

Add an extra style when no checks are present#656
mchehab wants to merge 1 commit into
getpatchwork:mainfrom
mchehab:no_checks

Conversation

@mchehab

@mchehab mchehab commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

On projects where all patches are meant to be checked, not having any checks mean that either the CI is still running or that CI didn't run for an specific patch.

At the first case, the maintainers need to be aware to wait for CI to run; at the last case, if it takes too long, they may need to manually re-run CI or double-check resuls.

Anyway, this is a sort of "error" status, so allow CSS to be able to customize it, by adding a "nochecks" state-like CSS style.

@mchehab

mchehab commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

For an example about its usecase, see: https://patchwork.linuxtv.org/project/linux-media/list/

@stephenfin

Copy link
Copy Markdown
Member

Can you fix the pep8 issue? You should be able to just run tox -e pep8 / pre-commit run -a.

On projects where all patches are meant to be checked, not
having any checks mean that either the CI is still running or
that CI didn't run for an specific patch.

At the first case, the maintainers need to be aware to wait for
CI to run; at the last case, if it takes too long, they may need
to manually re-run CI or double-check resuls.

Anyway, this is a sort of "error" status, so allow CSS to be
able to customize it, by adding a "nochecks" state-like CSS
style.

Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
@mchehab

mchehab commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

Can you fix the pep8 issue? You should be able to just run tox -e pep8 / pre-commit run -a.

Done.

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.

2 participants