Skip to content

fix: Fix new lint errors#1524

Merged
lehors merged 2 commits intoslsa-framework:mainfrom
lehors:fix-newlint-errors
Nov 24, 2025
Merged

fix: Fix new lint errors#1524
lehors merged 2 commits intoslsa-framework:mainfrom
lehors:fix-newlint-errors

Conversation

@lehors
Copy link
Member

@lehors lehors commented Nov 24, 2025

Merging PR #1522 updates the linter which leads to a bunch of new errors being reported.
This fixes those errors.

@netlify
Copy link

netlify bot commented Nov 24, 2025

Deploy Preview for slsa ready!

Name Link
🔨 Latest commit 27b2994
🔍 Latest deploy log https://app.netlify.com/projects/slsa/deploys/69249e74f4986200080d4971
😎 Deploy Preview https://deploy-preview-1524--slsa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lehors lehors changed the title Fix newlint errors fix: Fix newlint errors Nov 24, 2025
@lehors lehors requested a review from TomHennen November 24, 2025 16:51
@lehors lehors changed the title fix: Fix newlint errors fix: Fix new lint errors Nov 24, 2025
@lehors lehors force-pushed the fix-newlint-errors branch from b863af4 to b6fbaab Compare November 24, 2025 17:01
Copy link
Member

@arewm arewm left a comment

Choose a reason for hiding this comment

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

Thanks!

@lehors
Copy link
Member Author

lehors commented Nov 24, 2025

Hold it. I need to rebase and fix v1.2.

Signed-off-by: Arnaud J Le Hors <lehors@us.ibm.com>
Signed-off-by: Arnaud J Le Hors <lehors@us.ibm.com>
@lehors lehors force-pushed the fix-newlint-errors branch from b6fbaab to 27b2994 Compare November 24, 2025 18:05
@lehors lehors merged commit a034237 into slsa-framework:main Nov 24, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in Issue triage Nov 24, 2025
@lehors lehors deleted the fix-newlint-errors branch March 3, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants