Skip to content

Add return type hint to TZP.use - #1745

Open
chintakuntaharshavardhanreddy wants to merge 1 commit into
collective:mainfrom
chintakuntaharshavardhanreddy:add-tzp-use-type-hint
Open

Add return type hint to TZP.use#1745
chintakuntaharshavardhanreddy wants to merge 1 commit into
collective:mainfrom
chintakuntaharshavardhanreddy:add-tzp-use-type-hint

Conversation

@chintakuntaharshavardhanreddy

@chintakuntaharshavardhanreddy chintakuntaharshavardhanreddy commented Aug 31, 2026

Copy link
Copy Markdown

Linked issue

Description

Added the missing None return type hint to TZP.use().

The method does not return a value, so its return type is now explicitly annotated as None.

Checklist

  • I added a change log entry, following the instructions in all subsections under Change log requirements.
  • I followed icalendar's Artificial intelligence policy and disclosed my Responsible AI use in my commit messages, if applicable.
  • I added or updated tests, if applicable.
  • I ran and ensured all tests pass locally by following Run tests.
  • I added or edited documentation as necessary, both as docstrings to be rendered in the API documentation and narrative documentation, following the Style guide.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request did not pass quality checks and AI use is suspected. Please review Contribute and make any necessary amendments.

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

Copy link
Copy Markdown
Contributor

Profile summary:

GitHub user: chintakuntaharshavardhanreddy
🔴 Significant concerns found with user's profile.
🟢 No concerns found with recent PR activity.
🟢 No concerns found with recent issue activity.

For a more detailed report, run `gh-profiler chintakuntaharshavardhanreddy`.
Full profile
GitHub user: chintakuntaharshavardhanreddy
🔴 Significant concerns found with user's profile.
   🟢 Account age: 8 months
   🔴 No profile information provided.

🟢 No concerns found with recent PR activity.
   1 PR opened in the last 21 days.
      0 opened against repos the user owns.
      0 opened against repos in publicly associated orgs.
      1 opened against external repos.

   🟢 0 of 1 external PRs merged in the last 21 days.
   🟢 0 of 1 external PRs closed without merging in the last 21 days.

🟢 No concerns found with recent issue activity.
   🟢 No new issues opened in the last 21 days.

AI disclosure: I used GPT-5.6 Luna to assist with understanding the issue, reviewing the change, and guiding the contribution workflow.
@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 icalendar | 🛠️ Build #34315262 | 📁 Comparing f4e8880 against latest (59da8b8)

  🔍 Preview build  

2 files changed
± reference/api/icalendar.timezone.tzp.html
± _modules/icalendar/timezone/tzp.html

@angatha angatha left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your first contribution.

If you want to continue with other missing signatures, please chack all chackboxes in the PR template. That does indicate that you have done the step. E.g. " I added or updated tests, if applicable." does not ask you to add or update tests, but it also asks, that you checked it.

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