Skip to content

fix: docs linting#6452

Merged
LesnyRumcajs merged 4 commits intomainfrom
fix-linting
Jan 21, 2026
Merged

fix: docs linting#6452
LesnyRumcajs merged 4 commits intomainfrom
fix-linting

Conversation

@LesnyRumcajs
Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs commented Jan 20, 2026

Summary of changes

Changes introduced in this pull request:

  • expose some yarn tasks to mise
  • reuse those tasks in workflows
  • ensure proper formatting of auto-generated conformance reports

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Summary by CodeRabbit

  • Chores
    • Streamlined documentation CI by replacing several separate checks with unified lint and format tasks to speed and simplify validation.
    • Added an explicit documentation formatting step to CI to ensure docs are consistently formatted before signing/commit steps.
    • Simplified tooling initialization for doc tasks to reduce setup steps and improve workflow reliability.

✏️ Tip: You can customize this high-level summary in your review settings.

@LesnyRumcajs LesnyRumcajs requested a review from a team as a code owner January 20, 2026 12:51
@LesnyRumcajs LesnyRumcajs requested review from hanabi1224 and sudo-shashank and removed request for a team January 20, 2026 12:51
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 20, 2026

Walkthrough

Consolidates docs CI commands to use mise tasks: adds docs:format and docs:lint in mise.toml, replaces multiple yarn-based checks in .github/workflows/docs-check.yml with mise run docs:lint, and inserts mise run docs:format into .github/workflows/rpc-parity-report.yml.

Changes

Cohort / File(s) Summary
GitHub workflows
.github/workflows/docs-check.yml, .github/workflows/rpc-parity-report.yml
Replaced multiple yarn checks with a single mise run docs:lint in docs-check.yml; added mise run docs:format step to rpc-parity-report.yml.
Mise configuration
mise.toml
Added public tasks docs:format and docs:lint (description, dir, run); removed corepack enable from existing lint/format task initialization and simplified yarn invocation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • akaladarshi
  • hanabi1224
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly addresses the main objective of the PR: consolidating and improving docs linting by exposing yarn tasks to mise and reusing them in workflows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@LesnyRumcajs LesnyRumcajs changed the title Fix linting fix: docs linting Jan 20, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.76%. Comparing base (f96325d) to head (af457c6).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

see 8 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f96325d...af457c6. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue Jan 21, 2026
Merged via the queue into main with commit f8e4248 Jan 21, 2026
51 checks passed
@LesnyRumcajs LesnyRumcajs deleted the fix-linting branch January 21, 2026 10:26
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.

3 participants