Update CI/CD and readme#34
Conversation
- Clarified that target controls are disabled by default - Mark "New Scenes" repair flow as done - Move "Deleted Scenes" to its own task, and marked done - Removed Target repair flows (since there is nothing to do there, will be handled directly) - Removed Scene deduplication. Due to differences in how Scenes are handled by Tewke and entities are assigned to devices in HA, a better solution for the time being is implemented: allow users to disable Scenes during setup.)
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughAdds two validation workflows (HACS and hassfest), updates README feature entries and badge formatting, adds an issue_tracker to the tewke manifest, and removes the "new scenes" description from localization files. ChangesCI Workflows
README Documentation Update
Integration Manifest & Localization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/hassfest.yml:
- Line 14: The workflow currently uses a mutable reference "uses:
home-assistant/actions/hassfest@master"; replace this moving branch with a fixed
release tag or commit SHA (for example a specific version like `@vX.Y.Z` or a full
commit SHA) so the hassfest action is pinned and CI is reproducible and secure;
update the "uses: home-assistant/actions/hassfest@master" occurrence to the
chosen immutable tag/SHA.
- Line 13: The workflow currently pins the checkout action to
actions/checkout@v3 which uses Node 16; update the GitHub Actions step that
references "uses: \"actions/checkout@v3\"" to "uses: \"actions/checkout@v4\"" so
the job uses the supported major version (Node 20) and will run on current
GitHub-hosted runners.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 0c8e93c5-237a-440b-9710-65b9c0bf3f5e
📒 Files selected for processing (4)
.github/workflows/hassfest.ymlcustom_components/tewke/manifest.jsoncustom_components/tewke/strings.jsoncustom_components/tewke/translations/en.json
💤 Files with no reviewable changes (2)
- custom_components/tewke/translations/en.json
- custom_components/tewke/strings.json
✅ Files skipped from review due to trivial changes (1)
- custom_components/tewke/manifest.json
Readme list changes:
Part of ENG-1922
Summary by CodeRabbit
Documentation
Chores
Other