Skip to content

feat: adds --hideReportUrl flag to allow the users to hide the URL report creation #366.#380

Merged
v3nant merged 2 commits into
mainfrom
feat/hide-url
Oct 16, 2025
Merged

feat: adds --hideReportUrl flag to allow the users to hide the URL report creation #366.#380
v3nant merged 2 commits into
mainfrom
feat/hide-url

Conversation

@v3nant
Copy link
Copy Markdown
Contributor

@v3nant v3nant commented Oct 16, 2025 •
edited
Loading

  • Added --hideReportUrl flag to scan:eol

    • Let users (and CI runs) skip creating a public report URL per the new privacy requirement
    • Keeps console guidance clear by printing a save-your-report hint when the URL is hidden
    • Preserves analytics tracking by tagging scans with web_report_hidden

  • Added formatReportSaveHint helper and updated display flow

    • Centralizes the replacement messaging when the URL is suppressed

  • Regenerated the README to document the new flag

    • Ensures CLI help/usage examples reflect the additional option so users discover it easily

@v3nant v3nant marked this pull request as ready for review October 16, 2025 18:15
@v3nant v3nant requested a review from a team as a code owner October 16, 2025 18:15
marco-ippolito
marco-ippolito reviewed Oct 16, 2025
View reviewed changes
Comment thread src/service/display.svc.ts Outdated
marco-ippolito
marco-ippolito approved these changes Oct 16, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@marco-ippolito marco-ippolito left a comment

Choose a reason for hiding this comment

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

1 nit otherwise LGTM

@v3nant v3nant merged commit bf45226 into main Oct 16, 2025
12 checks passed
@v3nant v3nant deleted the feat/hide-url branch October 16, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeremymwells jeremymwells jeremymwells approved these changes

@marco-ippolito marco-ippolito marco-ippolito approved these changes

@facundo-herodevs facundo-herodevs facundo-herodevs approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@v3nant @jeremymwells @marco-ippolito @facundo-herodevs