Skip to content

ci: updated scorecard actions#6322

Merged
wesleytodd merged 1 commit intoexpressjs:masterfrom
Phillip9587:update-scorecard-actions
Feb 5, 2025
Merged

ci: updated scorecard actions#6322
wesleytodd merged 1 commit intoexpressjs:masterfrom
Phillip9587:update-scorecard-actions

Conversation

@Phillip9587
Copy link
Member

CI is still failing after #6314 because the scorecard workflow also uses the deprecated v3 version of actions/upload-artifact. I updated all commit shas of the scorecard workflow to their latest versions.

@Phillip9587
Copy link
Member Author

Hey @UlisesGascon, can we maybe fast-track this PR because the master branch checks are failing because of the scorecard workflow?

Copy link
Member

@wesleytodd wesleytodd left a comment

Choose a reason for hiding this comment

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

Checked the commits against the source repos, not that I don't trust you, but this is what we all need to do when we merge PRs like this to ensure the commits come from the source repo and not a fork.

@wesleytodd
Copy link
Member

I agree seems like we can fast track. Merging now.

@wesleytodd wesleytodd merged commit ae6a462 into expressjs:master Feb 5, 2025
21 checks passed
@bjohansebas bjohansebas mentioned this pull request Mar 15, 2025
@UlisesGascon UlisesGascon mentioned this pull request Mar 23, 2025
68 tasks
@wesleytodd wesleytodd mentioned this pull request Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants