Skip to content

ci: adopt Frogbot demo workflow improvements on v3.2.1 - #14

Merged
thst-nordic merged 1 commit into
mainfrom
ci/frogbot-demo-workflows
Jun 15, 2026
Merged

thst-nordic merged 1 commit into
mainfrom
ci/frogbot-demo-workflows

Conversation

@thst-nordic

Copy link
Copy Markdown
Contributor

Summary

Adopts workflow improvements from the JFrog Frogbot Demo, using jfrog/frogbot@v3.2.1:

  • frogbot-scan-pr.yml (replaces frogbot-scan-pull-request.yml)

    • Checks out the PR head SHA before scanning
    • Adds Node.js setup for dependency analysis
    • Sets JF_FAIL: "FALSE" so PR scans report findings without failing the check
  • frogbot-scan-and-fix.yml (replaces frogbot-scan-repository.yml)

    • Checks out the target branch before scanning
    • Adds Node.js setup
    • Keeps scheduled daily scans and JF_GIT_AGGREGATE_FIXES

Existing repo configuration is preserved: FROGBOT_URL / FROGBOT_ACCESS_TOKEN secrets, frogbot environment, and .frogbot/frogbot-config.yml.

Test plan

  • Merge and open a test PR; verify Frogbot scan runs after environment approval
  • Manually trigger Frogbot Scan and Fix workflow on main
  • Confirm findings are reported on PRs without failing the workflow (JF_FAIL: FALSE)

Replace the minimal Frogbot workflows with scan-pr and scan-and-fix
patterns from the JFrog demo: checkout PR/branch head, Node.js setup,
and JF_FAIL disabled for PR scans.
@thst-nordic
thst-nordic requested a review from a team as a code owner June 15, 2026 11:31
@github-actions

Copy link
Copy Markdown

👍 Frogbot scanned this pull request and did not find any new security issues.


@thst-nordic
thst-nordic merged commit fd36ae1 into main Jun 15, 2026
5 of 7 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.

1 participant