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.

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

Bypass summary printing if any of the group is pass-fail #236

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

siddharthkoli
Copy link

This PR removes the summary printing when --groupby flag contains pass-fail as one of the groups. Summary is only printed if pass-fail is absent.

Fixes #231

@siddharthkoli siddharthkoli changed the title Bypass summary printing if any of the group if pass-fail Bypass summary printing if any of the group is pass-fail Dec 21, 2024
@siddharthkoli siddharthkoli force-pushed the remove-passfail-summary branch from e90087b to 67d9373 Compare December 21, 2024 17:50
@kehoecj kehoecj added OSS Community Contribution Contributions from the OSS Community waiting-on-maintainer-review PR is waiting to be reviewed and functionally tested by the maintainers labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OSS Community Contribution Contributions from the OSS Community waiting-on-maintainer-review PR is waiting to be reviewed and functionally tested by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using the "pass-fail" --groupby option has reduntant summary output
3 participants