Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update shlint and add JSON lint yaml #1815

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Update shlint and add JSON lint yaml #1815

merged 1 commit into from
Sep 4, 2024

Conversation

rdlrt
Copy link
Contributor

@rdlrt rdlrt commented Sep 4, 2024

Description

  • Retire jormu scripts (previous commit is tagged)
  • Replace shlint (archived) with shellcheck github action
  • Add lint validation check for all branches, as well as add additional files: env, cntools.library and mithril.library

The additional files above may cause failure to run checks atm as these were not shell-checked before, tho I believe with severity: error - shouldnt be reported

  • Add JSON/YAML validator to github action

Retire jormu scripts (previous commit is tagged)
@rdlrt rdlrt requested a review from Scitz0 September 4, 2024 07:22
@rdlrt rdlrt merged commit ab22d82 into alpha Sep 4, 2024
4 checks passed
@rdlrt rdlrt deleted the shlint-upd branch September 4, 2024 07:36
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