Skip to content

Add ical_value property to vDate#1375

Closed
xxiaoxiong wants to merge 1 commit into
collective:mainfrom
xxiaoxiong:add-ical-value-vdate
Closed

Add ical_value property to vDate#1375
xxiaoxiong wants to merge 1 commit into
collective:mainfrom
xxiaoxiong:add-ical-value-vdate

Conversation

@xxiaoxiong
Copy link
Copy Markdown

This PR adds the ical_value property to vDate, contributing to Issue #876.

Changes

  • Added ical_value property to vDate (src/icalendar/prop/dt/date.py)
    • Returns: date - The Python date object
    • Type hint: date
    • RFC reference: :rfc:5545#section-3.3.4
    • Includes comprehensive docstring with examples

Tests

  • Created test_vDate.py with 5 comprehensive test cases:
    • test_ical_value_basic - Tests basic date return
    • test_ical_value_components - Tests year/month/day components
    • test_ical_value_from_ical - Tests with parsed ical strings
    • test_ical_value_leap_year - Tests leap year dates
    • test_ical_value_year_boundaries - Tests first/last day of year
  • All tests pass: 5 passed

Quality Checks

  • Add type hint
  • Add docstring with RFC reference
  • Add tests for all code paths
  • Pass ruff format check
  • Pass ruff lint check
  • All tests pass

Relates to #876

- Add ical_value property with type hints and docstring
- Returns the underlying date object
- Add comprehensive tests for ical_value (5 test cases)
- All tests pass
- Code formatted with ruff

Relates to collective#876
@read-the-docs-community
Copy link
Copy Markdown

Documentation build overview

📚 icalendar | 🛠️ Build #32600425 | 📁 Comparing 943c638 against latest (7357cec)

  🔍 Preview build  

5 files changed · ± 5 modified

± Modified

@stevepiercy stevepiercy added the ai-suspicion This contribution is possibly created with lots of AI help without enough human understanding. label May 20, 2026
@stevepiercy
Copy link
Copy Markdown
Member

@xxiaoxiong thank you for opening this PR. Because you haven't responded to the reviews and CI failures, I'm closing all of your PRs to let other contributors know that they are free to work on it. You're welcome to address our comments, resolve the matters, and reopen your pull request at any time, specifically to address the question of AI usage and add a change log, per Contribute. Thank you!

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.

2 participants