Skip to content

Add YAMLint YAML#40

Merged
sethvargo merged 1 commit intomainfrom
sethvargo/yamllint
Jul 17, 2025
Merged

Add YAMLint YAML#40
sethvargo merged 1 commit intomainfrom
sethvargo/yamllint

Conversation

@sethvargo
Copy link
Copy Markdown
Contributor

No description provided.

@sethvargo sethvargo requested a review from Copilot July 17, 2025 17:44
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a yamllint configuration file to enforce consistent YAML formatting and catch common errors

  • Introduces .yamllint.yml with a comprehensive set of style rules (indentation, spacing, quoting, etc.)
  • Defines directories to ignore (e.g., node_modules/, vendor/, thirdparty/)
Comments suppressed due to low confidence (3)

.yamllint.yml:1

  • [nitpick] Consider adding an extends: default at the top to inherit the upstream default rules and only override the ones you need, simplifying maintenance as yamllint evolves.
rules:

.yamllint.yml:1

  • [nitpick] Add a brief comment above this line (e.g., # YAML lint configuration for CI and local checks) explaining the file’s purpose and how to run yamllint.
rules:

.yamllint.yml:87

  • [nitpick] The ignore list includes both thirdparty/ and third_party/; if your repo only uses one naming style, consider removing the redundant pattern to reduce confusion.
  - 'thirdparty/'

@sethvargo sethvargo enabled auto-merge (squash) July 17, 2025 17:47
@sethvargo sethvargo merged commit ff59587 into main Jul 17, 2025
4 checks passed
@sethvargo sethvargo deleted the sethvargo/yamllint branch July 17, 2025 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants