Skip to content

YAML lint #110

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.

Sign up for GitHub

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

Jump to bottom
Merged
rimi-itk merged 2 commits into from
Jun 13, 2025
Merged

YAML lint #110

rimi-itk merged 2 commits into from
Jun 13, 2025

Conversation

@rimi-itk
Copy link
Contributor

@rimi-itk rimi-itk commented Jun 12, 2025 •
edited
Loading

  • Adds YAML linting
  • Lints YAML files – do we agree with how Prettier lints docker-compose*.yml files?

@rimi-itk rimi-itk force-pushed the feature/twig-lint branch from 33f2498 to fb829ac Compare June 12, 2025 13:21
@rimi-itk rimi-itk marked this pull request as ready for review June 12, 2025 15:48
@rimi-itk rimi-itk requested review from cableman and turegjorup June 12, 2025 15:48
@rimi-itk rimi-itk changed the title Twig lint YAML lint Jun 13, 2025
@turegjorup
Copy link
Contributor

turegjorup commented Jun 13, 2025

  • do we agree with how Prettier lints docker-compose*.yml files?

I have tried finding an official style guide without any luck. Personally I have no issue with how prettier lints yaml

@rimi-itk rimi-itk merged commit 26ebf6a into feature/github-actions-templates Jun 13, 2025
18 checks passed
@rimi-itk rimi-itk deleted the feature/twig-lint branch June 13, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cableman cableman cableman approved these changes

@turegjorup turegjorup turegjorup approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rimi-itk @turegjorup @cableman