Skip to content

chore: standardize markdownlint and yamllint configs - #134

Merged
gvatsal60 merged 1 commit into
masterfrom
chore/standardize-lint-configs
Oct 4, 2026
Merged

gvatsal60 merged 1 commit into
masterfrom
chore/standardize-lint-configs

Conversation

@gvatsal60

Copy link
Copy Markdown
Owner

This PR standardizes the markdownlint and yamllint configurations across the repo:

Changes:

  • .markdownlint.json: Standardized to use MD013 with line_length: 999 and MD033 with script as an allowed element
  • .yamllint.yaml: Added/ensured consistent line-length: max: 200 configuration

This ensures consistency across all repositories in the gvatsal60 org.

- Standardize .markdownlint.json with line_length: 999 and MD033 rule
- Ensure all repos have consistent .yamllint.yaml
Copilot AI balanced review requested due to automatic review settings October 4, 2026 06:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sonarqubecloud

sonarqubecloud Bot commented Oct 4, 2026

Copy link
Copy Markdown

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@gvatsal60
gvatsal60 merged commit e2974d1 into master Oct 4, 2026
12 checks passed
@gvatsal60
gvatsal60 deleted the chore/standardize-lint-configs branch October 4, 2026 07:07
@kilo-code-bot

kilo-code-bot Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • .markdownlint.json
Notes
  • Change is limited to .markdownlint.json; the .yamllint.yaml line-length: max: 200 item described in the PR body was already present in the base branch and is not part of this diff.
  • The file is valid JSON and both MD013.line_length and MD033.allowed_elements are valid markdownlint configuration keys. markdownlint-cli2 v0.23.3 (wired in .pre-commit-config.yaml) falls back to .markdownlint.json, so the config is picked up.
  • No Markdown file in the repo currently contains inline HTML, so the new MD033 allowance is inert today. It does allow raw <script> in future docs; worth keeping in mind if README content ever comes from untrusted sources.
  • No existing Kilo summary or inline comments were found for this PR, and no new issues were identified, so no inline comments were posted.

Reviewed by free · Input: 18.4K · Output: 3K · Cached: 190.5K

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants