Skip to content

Conversation

@kiwamizamurai
Copy link
Contributor

@kiwamizamurai kiwamizamurai commented Oct 17, 2025

resolved #237

Summary

  • Added Google yamlfmt v0.19.0 as a pre-commit hook to automatically format YAML files
  • Created .yamlfmt configuration file to define formatting rules
  • Applied formatting to all YAML files in the repository for consistency

Changes

  • Added yamlfmt to pre-commit hooks (.pre-commit-config.yaml)
  • Created yamlfmt configuration (.yamlfmt)
    • Retains line breaks for better readability
    • Sets max line length to 150 characters
    • Configures appropriate formatting options for the project

…ting

- Added YAML formatting hook with specific arguments for better consistency.
- Included a check for YAML files in test cases and helpers.
- Reformatted existing entries for clarity and consistency.
…d changes

- Added new entries in the changelog for breaking changes, dependencies, features, and fixes.
- Updated issue references in multiple YAML files to ensure tracking and documentation consistency.
- Enhanced the configuration of GitHub workflows and actions for better integration and testing.
- Included new test cases and fixtures to support recent changes in the MetricFlow project.
- Updated various YAML files to improve formatting and consistency, including changes to metric definitions and entity structures.
- Removed quotation marks from metric names and descriptions for uniformity.
- Adjusted the structure of YAML entries to align with project standards.
- Enhanced the changelog entry format for better readability.
- Introduced a new YAML formatter to enhance code quality and consistency in YAML files.
- Documented the change in the changelog with relevant details including author and issue reference.
@kiwamizamurai kiwamizamurai requested a review from a team as a code owner October 17, 2025 12:24
@cla-bot cla-bot bot added the cla:yes label Oct 17, 2025
- Refactored the pre-commit configuration to streamline hook definitions and improve clarity.
- Added a new YAML formatter configuration to enhance formatting consistency across YAML files.
- Ensured that the configuration adheres to project standards for better maintainability.
- Refactored various YAML files to ensure consistency and clarity in formatting.
- Added new entries and updated existing ones in the changelog for breaking changes, dependencies, features, and fixes.
- Enhanced test cases and fixtures to support recent changes in the MetricFlow project, ensuring better coverage and reliability.
- Improved the structure of YAML entries to align with project standards, facilitating easier maintenance and readability.
@kiwamizamurai
Copy link
Contributor Author

@courtneyholcomb
thank you always, please review this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add pre-commit YAML formatter

1 participant