search

TRI-ML / dgp

ML Dataset Governance Policy for Autonomous Vehicle Datasets
https://tri-ml.github.io/dgp/
MIT License
94 stars 62 forks source link

feat: replace last SuperLinter check `yamllint` with pre-commit hook #124

Closed tk-woven closed 1 year ago

tk-woven commented 1 year ago

Description

This concludes #114 . This introduces the yamllint pre-commit hook, runs it on all files, and touches up some violations. It also introduces a configuration for yamllint to ignore some checks that we don't need or otherwise modify them.

This is the last SuperLinter check to migrate, so this work also removes SuperLinter from CI.

This change is Reviewable

wadimkehl commented 1 year ago

Still unresolved conflicts?

tk-woven commented 1 year ago

Odd; there were some diffs in your recent merges to the conflicting files. Those diffs came from my previously-merged commits. Not sure what's going on here, but yes, more conflicts that I'll resolve!

tk-woven commented 1 year ago

I see, it was the combination of my changes reflowing some YAML and your changes bumping the Python version in them :)