feat: Add YAML formatter to pre-commit hooks (Issue #237) #1914
Open
kiwamizamurai wants to merge 7 commits intodbt-labs:mainfrom
Open
feat: Add YAML formatter to pre-commit hooks (Issue #237) #1914kiwamizamurai wants to merge 7 commits intodbt-labs:mainfrom
kiwamizamurai wants to merge 7 commits intodbt-labs:mainfrom
Conversation
…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.
- 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.
Contributor
Author
|
@courtneyholcomb |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
resolved #237
Summary
.yamlfmtconfiguration file to define formatting rulesChanges
.pre-commit-config.yaml).yamlfmt)