Skip to content

Add check/fix workflows for YAML files#364

Open
greenc-FNAL wants to merge 2 commits intomainfrom
maintenance/yaml-lint-prettier
Open

Add check/fix workflows for YAML files#364
greenc-FNAL wants to merge 2 commits intomainfrom
maintenance/yaml-lint-prettier

Conversation

@greenc-FNAL
Copy link
Contributor

No description provided.

@knoepfel
Copy link
Member

@greenc-FNAL, the number of checks has gotten very large. Why do we need a YAML check? The action will not run if there's a YAML error, right? Which seems like a chicken-and-egg problem.

@greenc-FNAL
Copy link
Contributor Author

greenc-FNAL commented Feb 25, 2026

Specifically for syntax errors in workflows, yes. It also enforces the usual style bits, like line length, unwanted trailing whitespace and indentation consistency.

There are several non-workflow YAML files in the repository rn (tool configurations), but I exepct there will be Phlex configs eventually, also.

These single language checks are quite fast (<10s) as long as we don't need a Phlex build environment to run them

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