Skip to content

Add an extra style when no checks are present - #656

Merged
stephenfin merged 1 commit into
getpatchwork:mainfrom
mchehab:no_checks
Oct 15, 2026
Merged

stephenfin merged 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.

@stephenfin
stephenfin added this pull request to the merge queue Oct 15, 2026
Merged via the queue into getpatchwork:main with commit 20e742e Oct 15, 2026
11 checks passed
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