Skip to content

Update publish-snapshot.yml#2303

Merged
priettt merged 1 commit intomainfrom
priettt/snapshotFix
Jul 3, 2025
Merged

Update publish-snapshot.yml#2303
priettt merged 1 commit intomainfrom
priettt/snapshotFix

Conversation

@priettt
Copy link
Copy Markdown
Contributor

@priettt priettt commented Jul 2, 2025

No description provided.

@priettt priettt requested a review from a team as a code owner July 2, 2025 19:41
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 2, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/checkout 4.*.* 🟢 5.2
Details
CheckScoreReason
Binary-Artifacts🟢 10no binaries found in the repo
Maintained⚠️ 01 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 0
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review🟢 10all changesets reviewed
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Packaging⚠️ -1packaging workflow not detected
Signed-Releases⚠️ -1no releases found
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Security-Policy🟢 9security policy file detected
Branch-Protection⚠️ -1internal error: error during GetBranch(releases/v2): error during branchesHandler.query: internal error: githubv4.Query: Resource not accessible by integration
SAST🟢 9SAST tool detected but not run on all commits
Vulnerabilities⚠️ 010 existing vulnerabilities detected

Scanned Files

  • .github/workflows/publish-snapshot.yml

Copy link
Copy Markdown
Contributor

@bidetofevil bidetofevil left a comment

Choose a reason for hiding this comment

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

Was this depending on the code already being on the machine when the workflow ran?

@priettt
Copy link
Copy Markdown
Contributor Author

priettt commented Jul 3, 2025

Was this depending on the code already being on the machine when the workflow ran?

yup, classic mistake 😅

@priettt priettt force-pushed the priettt/snapshotFix branch from 4008df5 to 21d9f56 Compare July 3, 2025 12:00
@priettt priettt merged commit d9a18ac into main Jul 3, 2025
13 checks passed
@priettt priettt deleted the priettt/snapshotFix branch July 3, 2025 12:40
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