Skip to content

Fix lint - #318

Merged
dragomirp merged 2 commits into
canonical:mainfrom
s-mlnk:lint-fix
Sep 2, 2026
Merged

dragomirp merged 2 commits into
canonical:mainfrom
s-mlnk:lint-fix

Conversation

@s-mlnk

@s-mlnk s-mlnk commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

CI is failing now due to lint checks because of the conflict between zizmor and actionlint. This PR pulls the new data platform workflows which fixes this issue by removing latter and formats workflow files to satisfy former.

@s-mlnk
s-mlnk marked this pull request as draft August 31, 2026 16:04
@s-mlnk
s-mlnk force-pushed the lint-fix branch 2 times, most recently from 85f6c3f to 288837c Compare August 31, 2026 16:50
@s-mlnk s-mlnk changed the title Fix lint. Fix lint Aug 31, 2026
@s-mlnk
s-mlnk marked this pull request as ready for review August 31, 2026 16:50
akram09
akram09 previously approved these changes Aug 31, 2026
@carlcsaposs-canonical

Copy link
Copy Markdown
Contributor

fyi we're planning to remove actionlint as soon as we have a second approval on canonical/data-platform-workflows#382

@akram09

akram09 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Thanks @carlcsaposs-canonical , @s-mlnk please bump lint workflow

@s-mlnk
s-mlnk requested a review from akram09 September 1, 2026 15:15
@s-mlnk

s-mlnk commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Done

@dragomirp
dragomirp merged commit 15c284a into canonical:main Sep 2, 2026
42 checks passed
@s-mlnk
s-mlnk deleted the lint-fix branch September 4, 2026 13:43
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.

4 participants