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.

Sign up for GitHub

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

Jump to bottom

HTML results status filter fix #5829

Merged
merged 1 commit into from
Dec 13, 2023
Merged

HTML results status filter fix #5829

merged 1 commit into from
Dec 13, 2023

Conversation

richtja
Copy link
Contributor

@richtja richtja commented Dec 11, 2023

This is an update of status filter in HTML results to correspond with the latest changes in a63423b, which introduced colored statuses.

Reference: #5828

@richtja
HTML results status filter fix
107e446 
This is an update of status filter in HTML results to correspond with
the latest changes in a63423b, which introduced colored statuses.

Reference: avocado-framework#5828
Signed-off-by: Jan Richter <jarichte@redhat.com>
@richtja richtja added the bug label Dec 11, 2023
@richtja richtja self-assigned this Dec 11, 2023
@richtja richtja linked an issue Dec 11, 2023 that may be closed by this pull request
The HTML report doesn't filter out tests by status #5828
Closed
@mr-avocado
Copy link

mr-avocado bot commented Dec 11, 2023

Dear contributor,
Avocado is currently under sprint #103, which is due to release an LTS (Long Term Stability) release.
Please avoid merging changes that do not fall into these categories:

  • Bug fixes
  • Usability Improvements
  • Documentation updates

As for the Avocado utility modules (“avocado.utils”) it is OK to introduce new functionality,
but changes to the existing APIs (including interface and behavior) should be avoided.
These kind of changes should wait until sprint #104.

@clebergnu
Copy link
Contributor

/packit copr-build

clebergnu
clebergnu approved these changes Dec 13, 2023
View reviewed changes
Copy link
Contributor

@clebergnu clebergnu left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

PS: the CI error is unrelated, and is being handled in #5830

@clebergnu clebergnu merged commit b152ad0 into avocado-framework:master Dec 13, 2023
64 of 65 checks passed
@pevogam
Copy link
Contributor

pevogam commented Dec 14, 2023

Thanks @richtja!

@richtja richtja deleted the html_select_fix branch December 14, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clebergnu clebergnu clebergnu approved these changes

Assignees

@richtja richtja

Labels
bug
Projects
Default project
Archived in project
Milestone
#103 (LTS release - Sound of Freedom)
Development

Successfully merging this pull request may close these issues.

The HTML report doesn't filter out tests by status
3 participants
@richtja @clebergnu @pevogam