-
Notifications
You must be signed in to change notification settings - Fork 1
Changelog #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changelog #6
Conversation
- Add GitHub Action to update changelog based on conventional commits - Create CONTRIBUTING.md with commit message guidelines - Add changelog section to README.md - Set up CHANGELOG.md with proper format
Summary of ChangesHello @brombomb, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request establishes a robust and automated changelog management system for the project, particularly for Home Assistant updates. It introduces a structured Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request introduces a changelog generation process based on Conventional Commits. It adds a CHANGELOG.md file, a CONTRIBUTING.md with commit guidelines, and updates the README.md to reference these new files. The changes are a great step towards automating release notes and improving contributor experience. I've provided a few suggestions to enhance consistency and fix some minor issues in the new documentation files.
| ### Added | ||
| - Initial release of TronbytAssistant integration | ||
| - Support for Tronbyt device notifications from Home Assistant | ||
| - Brightness control via light entity | ||
| - Update interval configuration via number entity | ||
| - Pinned app selection via select entity | ||
| - Night mode toggle via switch entity | ||
| - Night mode start/end time configuration | ||
| - Night mode brightness and app selection | ||
| - Dim mode start time and brightness configuration | ||
| - Built-in notification service (Push) | ||
| - Custom text notification service (Text) | ||
| - App deletion service (Delete) | ||
| - Template support for dynamic content | ||
| - Argument passing to apps | ||
| - HACS integration support | ||
| - Configuration flow for easy setup | ||
| - Multi-language support (English and German translations) | ||
|
|
||
| ### Features | ||
| - Local polling IoT class for reliable communication | ||
| - Single config entry design | ||
| - Multi-device support | ||
| - Service selectors that update automatically | ||
| - Template value support for dynamic content | ||
| - HTTP error handling and logging | ||
| - Pre-commit hooks for code quality | ||
| - Comprehensive test suite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CHANGELOG.md file states it's based on Keep a Changelog, but it uses a non-standard Features section. To adhere to the standard, the items under Features should be merged into the Added section, as they represent new functionality added in that release.
### Added
- Initial release of TronbytAssistant integration
- Support for Tronbyt device notifications from Home Assistant
- Brightness control via light entity
- Update interval configuration via number entity
- Pinned app selection via select entity
- Night mode toggle via switch entity
- Night mode start/end time configuration
- Night mode brightness and app selection
- Dim mode start time and brightness configuration
- Built-in notification service (Push)
- Custom text notification service (Text)
- App deletion service (Delete)
- Template support for dynamic content
- Argument passing to apps
- HACS integration support
- Configuration flow for easy setup
- Multi-language support (English and German translations)
- Local polling IoT class for reliable communication
- Single config entry design
- Multi-device support
- Service selectors that update automatically
- Template value support for dynamic content
- HTTP error handling and logging
- Pre-commit hooks for code quality
- Comprehensive test suite|
|
||
| ## Manual Workflow Trigger | ||
|
|
||
| You can also manually trigger the changelog update by going to the Actions tab and running the "Update Changelog" workflow. No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a good practice to end text files with a newline character. This prevents issues with some command-line tools and source control systems. Please add a newline at the end of this file.
| You can also manually trigger the changelog update by going to the Actions tab and running the "Update Changelog" workflow. | |
| You can also manually trigger the changelog update by going to the Actions tab and running the "Update Changelog" workflow. | |
| - Development setup | ||
| - Pull request process |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a changelog generator for the HA updates.