Repository navigation
Add ical_value property to vTime - #2105
wmortenson199 wants to merge 2 commits into
Conversation
|
Thanks for the pull request, @wmortenson199! A maintainer will review it shortly. |
|
One or more PR-quality checks did not pass. Review the PR Quality Checks summary in the workflow run for the failed checks and their details, then make any necessary updates. Checklist items may remain unchecked while work is in progress. Please tick each item as you complete it. If you used AI assistance, follow the project's Responsible AI use policy and provide the required disclosures. |
Insufficient dataNot enough activity yet to make a reliable assessment. This is an automated analysis by AgentScan |
Part of PyCalendar#876. AI: Claude Opus 5.5 Prompt (summary): help draft an ical_value property for vTime following vDate.ical_value. Output: The property, docstring, tests, and news entry in this commit. I reviewed the code, placed the files, fixed the test file location, and ran the full test suite locally.
d1843d3 to
cc08ea5
Compare
|
Thanks for the automated check. I've shortened the description, updates the bio on my profile, and updated the commit message and news entry to fully disclose AI use per the project's AI policy. Happy to make any other changes. |
Linked issue
Description
Adds the Ical_value property to vTime that returns the time stored. Followed the syntax used for the vDate portion. Changes include twp tests and a news entry.
AI disclosure: Claude Opus 5.5 helped verify the property, tests, and news entry after I drafted the code. Also helped pluck how other changes in different instances were implemented. Claude made changes to my original code, I then reviewed and tested the code locally.
Additional information
This is my first contribution to icalendar, made as part of a university open source course. All tests pass locally (
uv run pytest).