Skip to content

Add ical_value property to vTime - #2105

Open
wmortenson199 wants to merge 2 commits into
PyCalendar:mainfrom
Dahshan228:ical-value-vtime
Open

wmortenson199 wants to merge 2 commits into
PyCalendar:mainfrom
Dahshan228:ical-value-vtime

Conversation

@wmortenson199

@wmortenson199 wmortenson199 commented Nov 12, 2026 •

Copy link
Copy Markdown

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).

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the pull request, @wmortenson199! A maintainer will review it shortly.

@github-actions

Copy link
Copy Markdown
Contributor

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.

@github-actions github-actions Bot added the ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding. label Nov 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Insufficient data

Not enough activity yet to make a reliable assessment.

View full analysis →

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.
@wmortenson199

Copy link
Copy Markdown
Author

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.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant