Skip to content

fix: scorecard workflow#896

Closed
niltonpimentel02 wants to merge 1 commit into
django-commons:mainfrom
niltonpimentel02:895
Closed

fix: scorecard workflow#896
niltonpimentel02 wants to merge 1 commit into
django-commons:mainfrom
niltonpimentel02:895

Conversation

@niltonpimentel02

Copy link
Copy Markdown
Member

Remove the unexpected key from the workflow.

Closes #895

Copilot AI review requested due to automatic review settings May 24, 2026 00:41
@niltonpimentel02 niltonpimentel02 requested a review from bckohan as a code owner May 24, 2026 00:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes the OpenSSF Scorecard GitHub Actions workflow by removing an invalid key in the environment section that was causing workflow parsing/verification to fail (per issue #895).

Changes:

  • Removed deployment: false from the job environment block to comply with GitHub Actions workflow schema.
  • Restores Scorecard workflow validity so results can be sent to the Scorecard webapp without a 400 parse error.

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

@bckohan

bckohan commented May 25, 2026

Copy link
Copy Markdown
Member

This is actually a problem with the ossf/scorecard-webapp. The deployment key is a new way to tell github to not generate a deployment for every action that uses and environment. We use environments to protect secrets with special access rules to harden the CI.

I'm just letting scorecard be broken until the update their dependencies.

@bckohan bckohan closed this May 25, 2026
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.

Scorecard analysis workflow breaking

3 participants