Skip to content

Wait for Gradle wrapper validation before Scorecard - #1242

Closed
opentelemetrybot wants to merge 1 commit into
open-telemetry:mainfrom
opentelemetrybot:scorecard-polling/java-examples-2026-08-07
Closed

Wait for Gradle wrapper validation before Scorecard#1242
opentelemetrybot wants to merge 1 commit into
open-telemetry:mainfrom
opentelemetrybot:scorecard-polling/java-examples-2026-08-07

Conversation

@opentelemetrybot

Copy link
Copy Markdown
Contributor

Wait for the Gradle wrapper validation push run at the exact commit before invoking the reusable OSSF Scorecard workflow. This prevents the validated wrapper JAR from appearing intermittently as a binary artifact while preserving the existing Scorecard workflow, schedule, publication, filtering, and failure notification. Qualified on run 31199311175.

@trask
trask requested a lite review from Copilot August 7, 2026 16:51
@trask trask closed this Aug 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the OSSF Scorecard GitHub Actions workflow to reliably wait for the repository’s Gradle wrapper validation workflow run (for the same commit SHA) to complete successfully before invoking the reusable Scorecard workflow, preventing intermittent wrapper JAR artifacts from being observed during Scorecard execution.

Changes:

  • Added a wait-for-gradle-wrapper-validation job that polls GitHub Actions for a successful gradle-wrapper-validation.yml run matching the current github.sha.
  • Made the analysis (Scorecard) job depend on the new wait job via needs, preserving the existing Scorecard reusable workflow invocation and notification behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants