Skip to content

Add yaml linter check#173

Merged
kr-t merged 4 commits intomainfrom
add-yml-linter
Feb 19, 2026
Merged

Add yaml linter check#173
kr-t merged 4 commits intomainfrom
add-yml-linter

Conversation

@SorinOlari
Copy link
Collaborator

@SorinOlari SorinOlari commented Feb 17, 2026 •
edited
Loading

Description

Added YAML linter to the project and integrated it into the CI pipeline. All detected formatting issues have been fixed.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code cleanup/refactoring
  • CI system update

How Has This Been Tested?

Intentionally introduced YAML formatting issues to ensure the linter detects them as expected

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@SorinOlari SorinOlari self-assigned this Feb 17, 2026
@SorinOlari SorinOlari marked this pull request as draft February 17, 2026 12:46
@SorinOlari SorinOlari changed the title Add yml linter check [DRAFT] Add yml linter check Feb 17, 2026
@SorinOlari SorinOlari force-pushed the add-yml-linter branch 2 times, most recently from b0e4e18 to 9ee86f2 Compare February 17, 2026 16:27
@SorinOlari SorinOlari changed the title [DRAFT] Add yml linter check Add yaml linter check Feb 18, 2026
@SorinOlari SorinOlari linked an issue Feb 18, 2026 that may be closed by this pull request
Add YAML formatter #166
Closed
@SorinOlari SorinOlari added this to Ocre Feb 18, 2026
@SorinOlari SorinOlari marked this pull request as ready for review February 18, 2026 10:20
@SorinOlari SorinOlari requested a review from casaroli February 18, 2026 10:20
casaroli
casaroli reviewed Feb 18, 2026
View reviewed changes
casaroli
casaroli reviewed Feb 18, 2026
View reviewed changes
casaroli
casaroli previously approved these changes Feb 18, 2026
View reviewed changes
casaroli
casaroli previously approved these changes Feb 18, 2026
View reviewed changes
@SorinOlari @kr-t
scripts: add yml checker script
f78fb11 
Signed-off-by: SorinO <sorin.olari2@gmail.com>
@SorinOlari @kr-t
CI(formatting-checks): add yaml format checks to runner
416a453 
Signed-off-by: SorinO <sorin.olari2@gmail.com>
@SorinOlari @kr-t
CI(formatting-checks): use sh instead of bash to ensure compatibility…
d99eefa 
… (BASH_SOURCE only in bash)

Signed-off-by: SorinO <sorin.olari2@gmail.com>
@SorinOlari @kr-t
CI: fix errors detected by yml linter
70d32ef 
Signed-off-by: SorinO <sorin.olari2@gmail.com>
@kr-t kr-t merged commit 70d32ef into main Feb 19, 2026
3 checks passed
@github-project-automation github-project-automation bot moved this to Done in Ocre Feb 19, 2026
@kr-t kr-t deleted the add-yml-linter branch February 19, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kr-t kr-t kr-t approved these changes

@casaroli casaroli casaroli left review comments

@srberard srberard Awaiting requested review from srberard srberard is a code owner

Assignees

@SorinOlari SorinOlari

Labels

None yet

Projects

Ocre
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add YAML formatter

3 participants

@SorinOlari @casaroli @kr-t