Skip to content

Conversation

@courtneypacheco
Copy link
Contributor

@courtneypacheco courtneypacheco commented Mar 6, 2025

Issue resolved by this Pull Request:
Resolves #5

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been added and/or updated, if applicable.
  • Unit tests have been added and/or updated. (If this is not applicable, please provide a justification.)
  • Integration testing has been performed, if applicable

Description of this Change

We should add actionlint to this repository so that all GitHub workflow and in-house action files get validated.

Note: all of these "actionlint" files were copied directly from: https://github.com/instructlab/instructlab/blob/main/.github/workflows/

The main difference is that I removed the "install dependent PRs" portion of actionlint because it's not applicable to this use case.

@mergify mergify bot added the CI/CD Affects CI/CD configuration label Mar 6, 2025
@courtneypacheco courtneypacheco marked this pull request as ready for review March 6, 2025 19:06
@courtneypacheco courtneypacheco requested a review from ktdreyer March 6, 2025 19:06
@courtneypacheco courtneypacheco added the chore You gotta do what you gotta do label Mar 6, 2025
This linter should validate GitHub workflow files as well as in-house GH action files.

Signed-off-by: Courtney Pacheco <6019922+courtneypacheco@users.noreply.github.com>
@kami619
Copy link

kami619 commented Mar 6, 2025

The PR looks good to me.

@mergify mergify bot added the one-approval label Mar 6, 2025
@ktdreyer
Copy link
Contributor

ktdreyer commented Mar 6, 2025

I'll merge this with Kamesh's review

@ktdreyer ktdreyer merged commit 857e83f into main Mar 6, 2025
6 checks passed
@courtneypacheco courtneypacheco deleted the add-actionlint branch March 6, 2025 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore You gotta do what you gotta do CI/CD Affects CI/CD configuration one-approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Add actionlint to check GitHub Action workflow files

4 participants