-
Notifications
You must be signed in to change notification settings - Fork 0
Enhancements #21
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
Enhancements #21
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #21 +/- ##
==========================================
- Coverage 97.30% 97.28% -0.02%
==========================================
Files 15 15
Lines 2151 2139 -12
==========================================
- Hits 2093 2081 -12
Misses 58 58 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Pull Request Overview
This PR introduces enhancements to the calendar library, focusing on code refactoring and a new utility function.
- Added a new
parseBooleanfunction to parse boolean property values from calendar properties - Refactored occurrence query methods to use component-level
occurrencesmethods instead of duplicating RecurrenceIterator initialization logic
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| lib/src/semantic/property_parsers.dart | Adds parseBoolean helper function to parse boolean values from calendar properties |
| lib/src/semantic/extensions/queries.dart | Refactors occurrence queries for EventComponent, TodoComponent, JournalComponent, and TimeZoneSubComponent to delegate to component-level occurrences methods, eliminating code duplication |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull Request Overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.