Skip to content

Add timedelta as valid variable type#3084

Open
cnotin wants to merge 1 commit intohome-assistant:masterfrom
cnotin:patch-1
Open

Add timedelta as valid variable type#3084
cnotin wants to merge 1 commit intohome-assistant:masterfrom
cnotin:patch-1

Conversation

@cnotin
Copy link
Copy Markdown

@cnotin cnotin commented May 4, 2026

Proposed change

timedelta should be a valid variable type. I don't know why it wasn't so I let you double-check in case there's a bigger issue I'm not aware of.

Separate PR, to actually add it: home-assistant/home-assistant.io#45180

Type of change

Checklist

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

Additional information

Separate PR, to actually add it, is coming
@cnotin cnotin marked this pull request as draft May 4, 2026 23:05
@cnotin cnotin marked this pull request as ready for review May 4, 2026 23:13
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 4, 2026

📝 Walkthrough

Walkthrough

A documentation page is updated to clarify the type: configuration key, specifying allowed values, multi-type syntax, and validation constraints for map/list and boolean types.

Changes

Documentation Update

Layer / File(s) Summary
Available Keys Documentation
docs/documenting/create-page.md
The type: bullet entry is clarified to document allowed type values (map, list, selector, any), multi-type syntax (e.g., [string, integer]), and validation constraints: map/list require keys:, and boolean requires default:.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding timedelta as a valid variable type to the documentation.
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.
Description check ✅ Passed The PR description follows the template structure with all required sections completed: proposed change, type of change (with single selection), checklist items marked, and additional information provided.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

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