Skip to content

Pin dependencies#3080

Closed
silamon wants to merge 1 commit into
masterfrom
fixup-deps
Closed

Pin dependencies#3080
silamon wants to merge 1 commit into
masterfrom
fixup-deps

Conversation

@silamon

@silamon silamon commented May 1, 2026

Copy link
Copy Markdown
Contributor

Proposed change

Type of change

  • Document existing features within Home Assistant
  • Document new or changing features for which there is an existing pull request elsewhere
  • Spelling or grammatical corrections, or rewording for improved clarity
  • Changes to the backend of this documentation
  • Remove stale or deprecated documentation

Checklist

  • I have read and followed the documentation guidelines.
  • I have verified that my changes render correctly in the documentation.

Additional information

  • This PR fixes or closes issue: fixes #
  • Link to relevant existing code or pull request:

Summary by CodeRabbit

  • Chores
    • Pinned dependency versions to exact versions for improved build stability and reproducibility.
    • Raised minimum Node.js engine requirement to version 20.0 or higher.

@coderabbitai

coderabbitai Bot commented May 1, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The pull request tightens dependency version constraints in package.json by replacing caret-based semver ranges with exact version pins for multiple packages (Docusaurus, React, ReactDOM, and related tools) and raises the Node.js engine requirement from >=18.0 to >=20.0.

Changes

Cohort / File(s) Summary
Package Configuration
package.json
Pinned exact versions for 9 dependencies (@docusaurus/core, @docusaurus/preset-classic, @docusaurus/theme-mermaid, @mdx-js/react, @easyops-cn/docusaurus-search-local, by-node-env, clsx, react, react-dom, yaml-loader) by removing caret (^) ranges. Increased Node.js engine requirement from >=18.0 to >=20.0.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is entirely empty with only the template structure present; no actual content was provided in any required sections like 'Proposed change' or type of change selection. Fill in the 'Proposed change' section explaining the dependency updates, select the appropriate 'Type of change' checkbox, and complete all required checklist items and additional information.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title 'Pin dependencies' directly and clearly summarizes the main change: converting from caret-based semver ranges to exact version pins in package.json.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fixup-deps

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@silamon silamon changed the title Fix dependencies Unpin dependencies May 1, 2026
@silamon silamon changed the title Unpin dependencies Pin dependencies May 1, 2026
@silamon

silamon commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Will push this with a dependency change instead of waiting. :)

@silamon silamon closed this Jul 10, 2026
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.

1 participant