Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor updates in support of #569 #597

Conversation

patrickmckenna
Copy link

@sdispater given your obligations with Poetry, I thought it might help to open a fresh PR against #569, with the hope of nudging those massive, and eagerly anticipated(!), updates closer to release 😄. Hopefully this helps, but if not, please feel free to close/ignore/request further changes as you see fit.

Given the wide scope of that PR, the best way to help seemed to be taking the checklist in #569 (comment) at face value, i.e. assume the needed tests are written, and look into what docs updates are still needed. But before starting on that, it seemed like a few other minor updates were necessary:

  • Merge the latest updates from master
  • Switch to using a somewhat more stable version of upstream poetry (1745659). I couldn't get this project up and running locally using Poetry 1.1, and I'm guessing that's a known issue—hence all the build scripts previously pointing to a newer, unreleased ref.
  • Updating the README and build/release scripts to reference the right Python versions, based on 0c5b78e and 1102725.

c44bceb was a temporary hack; I'll revert it.

Please LMK if this is on the right track. As far as substantive doc edits go, do you have a clear idea of what the minimum viable updates are? I'd guess that support for zoneinfo TZs is among the most important changes to explain, but what other topics would you include?

Hope this helps!

/cc @NickFabry @anvit25 (per #590 (comment))

@patrickmckenna patrickmckenna changed the title Patrickmckenna/modernize codebase Minor updates in support of #569 Jan 4, 2022
@patrickmckenna patrickmckenna force-pushed the patrickmckenna/modernize-codebase branch from 673b9db to aba8764 Compare January 4, 2022 04:01
@patrickmckenna patrickmckenna force-pushed the patrickmckenna/modernize-codebase branch from aba8764 to f5c40de Compare January 4, 2022 04:04
@patrickmckenna
Copy link
Author

FWIW, the tests are passing.

They map 1-1, but the previous implementation turned ZoneInfo objs into
FixedTimezone objs, which they generally aren't.
@sdispater sdispater force-pushed the modernize-codebase branch 2 times, most recently from d501c47 to 607bab4 Compare January 18, 2022 23:07
@sdispater sdispater deleted the branch python-pendulum:modernize-codebase January 18, 2022 23:23
@sdispater sdispater closed this Jan 18, 2022
@patrickmckenna patrickmckenna deleted the patrickmckenna/modernize-codebase branch January 20, 2022 19:39
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