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

Issue #101: As a Devops, I would like to have a unified set of rules for yaml linting #102

Conversation

SonOfLope
Copy link
Contributor

@SonOfLope SonOfLope commented Mar 1, 2024

To provide conformity, we are adding a config yaml linting file that provides expected standards of yaml files for the organisation. Repositories using the reusable workflow for linting yaml files can simply omit to specify a config file path to use default config of the organisation.

Also fixes ai-cfia/howard#90

@SonOfLope SonOfLope requested a review from a team March 1, 2024 21:45
Copy link
Contributor

@rngadam rngadam left a comment

Choose a reason for hiding this comment

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

using curl to download file is not necessary; use uses keyword instead to ensure that the .yamllint.yml file is present.

.github/workflows/workflow-yaml-check.yml Outdated Show resolved Hide resolved
@SonOfLope SonOfLope merged commit e3fdb01 into main Mar 6, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
2 participants