Skip to content

MPT-22665 Skip PR Teams notification for Dependabot#75

Merged
ffaraone merged 1 commit into
mainfrom
bugfix/MPT-22665-skip-teams-notify-on-dependabot
Jun 26, 2026
Merged

MPT-22665 Skip PR Teams notification for Dependabot#75
ffaraone merged 1 commit into
mainfrom
bugfix/MPT-22665-skip-teams-notify-on-dependabot

Conversation

@d3rky

@d3rky d3rky commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

🤖 AI-generated PR — Please review carefully.

What

Guard the two PR-notification steps in the build job (pr-build-merge.yaml) with && github.actor != 'dependabot[bot]':

  • "Compute added/removed lines for notification"
  • "Notify Microsoft Teams"

Why

On Dependabot PRs secrets.TEAMS_WEBHOOK_URL is empty (Dependabot has no access to repository secrets), so the notify action receives an empty webhook URL and curl fails with URL rejected: Malformed input to a URL function — failing the whole build job even though tests pass. This currently blocks every Dependabot PR (#68#74). Real PRs are unaffected and still post the notification.

Testing

Workflow YAML validated; diff is limited to the two if: conditions.

The build job's "Notify Microsoft Teams" step uses secrets.TEAMS_WEBHOOK_URL,
which is empty on Dependabot PRs (Dependabot has no secret access), so curl
fails with a malformed URL and the build job goes red on every Dependabot PR.
Guard the notification steps (and the diff computation feeding them) with
github.actor != 'dependabot[bot]' so Dependabot PRs no longer fail build.

MPT-22665
@sonarqubecloud

Copy link
Copy Markdown

@ffaraone ffaraone merged commit eb2ee33 into main Jun 26, 2026
8 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.

2 participants