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

Hide titles if the tags mode is enabled #32

Merged
merged 2 commits into from
Feb 25, 2023
Merged

Hide titles if the tags mode is enabled #32

merged 2 commits into from
Feb 25, 2023

Conversation

KoolTheba
Copy link
Collaborator

@KoolTheba KoolTheba commented Feb 24, 2023 •
edited
Loading

Main changes

  • Hide title and subtitle in the report template if the tag mode is enabled
  • Added unit test for covering all cases related to the utility generateReportContent

Context

Changelog

  • ef54276 feat: hidden template title and subtitle if reportTags flag is enabled by @KoolTheba
  • 611de75 test: added unit test for generateReportContent utility by @KoolTheba

@UlisesGascon UlisesGascon added the enhancement New feature or request label Feb 24, 2023
@UlisesGascon UlisesGascon added this to the v2.0.0-beta milestone Feb 24, 2023
@KoolTheba KoolTheba mentioned this pull request Feb 24, 2023
Open
3 tasks
@UlisesGascon
Copy link
Member

UlisesGascon commented Feb 25, 2023 •
edited
Loading

@KoolTheba Thanks for the patch and the tests!

@UlisesGascon UlisesGascon merged commit e3526f9 into ossf:main Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@UlisesGascon UlisesGascon UlisesGascon approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
v2.0.0-beta
Development

Successfully merging this pull request may close these issues.

Hide titles if the tags mode is enabled
2 participants
@KoolTheba @UlisesGascon