gh-103822: change return value to enum for day and month APIs by Agent-Hellboy · Pull Request #103827 · python/cpython · GitHub
Skip to content

gh-103822: change return value to enum for day and month APIs#103827

Merged
ethanfurman merged 3 commits intopython:mainfrom
Agent-Hellboy:fix-issue-103822
May 2, 2023
Merged

gh-103822: change return value to enum for day and month APIs#103827
ethanfurman merged 3 commits intopython:mainfrom
Agent-Hellboy:fix-issue-103822

Conversation

@Agent-Hellboy
Copy link
Copy Markdown
Contributor

@Agent-Hellboy Agent-Hellboy commented Apr 25, 2023

this is an initial commit to get some quick feedback on calendar APIs which I can change

@arhadthedev arhadthedev added the stdlib Standard Library Python modules in the Lib/ directory label Apr 25, 2023
@Agent-Hellboy Agent-Hellboy changed the title gh-103822: change return value to enum for day and month API in gh-103822: change return value to enum for day and month APIs Apr 29, 2023
Copy link
Copy Markdown
Member

@ethanfurman ethanfurman left a comment

Choose a reason for hiding this comment

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

I thought there would be more places in the file where Day and Month would be used. Oh, well.

Make those two changes and we'll call it good.

Comment thread Lib/calendar.py Outdated
Comment thread Lib/calendar.py Outdated
@bedevere-bot
Copy link
Copy Markdown

@Agent-Hellboy
Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again

@bedevere-bot
Copy link
Copy Markdown

@bedevere-bot bedevere-bot requested a review from ethanfurman May 2, 2023 04:53
@ethanfurman ethanfurman merged commit 1f53844 into python:main May 2, 2023
carljm added a commit to carljm/cpython that referenced this pull request May 2, 2023
* main:
  pythongh-103822: [Calendar] change return value to enum for day and month APIs (pythonGH-103827)
  pythongh-65022: Fix description of tuple return value in copyreg (python#103892)
  pythonGH-103525: Improve exception message from `pathlib.PurePath()` (pythonGH-103526)
  pythongh-84436: Add integration C API tests for immortal objects (pythongh-103962)
  pythongh-103743: Add PyUnstable_Object_GC_NewWithExtraData (pythonGH-103744)
  pythongh-102997: Update Windows installer to SQLite 3.41.2. (python#102999)
  pythonGH-103484: Fix redirected permanently URLs (python#104001)
  Improve assert_type phrasing (python#104081)
  pythongh-102997: Update macOS installer to SQLite 3.41.2. (pythonGH-102998)
  pythonGH-103472: close response in HTTPConnection._tunnel (python#103473)
  pythongh-88496: IDLE - fix another test on macOS (python#104075)
  pythongh-94673: Hide Objects in PyTypeObject Behind Accessors (pythongh-104074)
  pythongh-94673: Properly Initialize and Finalize Static Builtin Types for Each Interpreter (pythongh-104072)
  pythongh-104016: Skip test for deeply neste f-strings on wasi (python#104071)
carljm added a commit to carljm/cpython that referenced this pull request May 2, 2023
* main: (760 commits)
  pythonGH-104102: Optimize `pathlib.Path.glob()` handling of `../` pattern segments (pythonGH-104103)
  pythonGH-104104: Optimize `pathlib.Path.glob()` by avoiding repeated calls to `os.path.normcase()` (pythonGH-104105)
  pythongh-103822: [Calendar] change return value to enum for day and month APIs (pythonGH-103827)
  pythongh-65022: Fix description of tuple return value in copyreg (python#103892)
  pythonGH-103525: Improve exception message from `pathlib.PurePath()` (pythonGH-103526)
  pythongh-84436: Add integration C API tests for immortal objects (pythongh-103962)
  pythongh-103743: Add PyUnstable_Object_GC_NewWithExtraData (pythonGH-103744)
  pythongh-102997: Update Windows installer to SQLite 3.41.2. (python#102999)
  pythonGH-103484: Fix redirected permanently URLs (python#104001)
  Improve assert_type phrasing (python#104081)
  pythongh-102997: Update macOS installer to SQLite 3.41.2. (pythonGH-102998)
  pythonGH-103472: close response in HTTPConnection._tunnel (python#103473)
  pythongh-88496: IDLE - fix another test on macOS (python#104075)
  pythongh-94673: Hide Objects in PyTypeObject Behind Accessors (pythongh-104074)
  pythongh-94673: Properly Initialize and Finalize Static Builtin Types for Each Interpreter (pythongh-104072)
  pythongh-104016: Skip test for deeply neste f-strings on wasi (python#104071)
  pythongh-104057: Fix direct invocation of test_super (python#104064)
  pythongh-87092: Expose assembler to unit tests (python#103988)
  pythongh-97696: asyncio eager tasks factory (python#102853)
  pythongh-84436: Immortalize in _PyStructSequence_InitBuiltinWithFlags() (pythongh-104054)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stdlib Standard Library Python modules in the Lib/ directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants