Skip to content

drop python2, add typing hint, rename naming to follow pep8#22

Open
chuiniudawang wants to merge 4 commits into
lidaobing:masterfrom
chuiniudawang:master
Open

drop python2, add typing hint, rename naming to follow pep8#22
chuiniudawang wants to merge 4 commits into
lidaobing:masterfrom
chuiniudawang:master

Conversation

@chuiniudawang

Copy link
Copy Markdown

No description provided.

Comment thread AGENTS.md Outdated
@@ -0,0 +1,41 @@
# python-lunardate

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

remove this file

Comment thread lunardate.py

@staticmethod
def leapMonthForYear(year):
def leap_month_for_year(year: int) -> Optional[int]:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

keep it backword compatible

Comment thread README.md Outdated

* 0.2.2: add LunarDate.leapMonthForYear; fix bug in year 1899
* 0.2.3: rename all APIs to PEP 8 snake_case; add type hints; drop Python 2 support
* 0.2.2: add LunarDate.leap_month_for_year; fix bug in year 1899

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Don't change the old NEWS

Comment thread README.md Outdated
## News

* 0.2.2: add LunarDate.leapMonthForYear; fix bug in year 1899
* 0.2.3: rename all APIs to PEP 8 snake_case; add type hints; drop Python 2 support

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Don't announce 0.2.3

@chuiniudawang chuiniudawang requested a review from lidaobing July 5, 2026 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants