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

Use tomllib/tomli over toml #284

Merged
merged 2 commits into from
May 1, 2023
Merged

Use tomllib/tomli over toml #284

merged 2 commits into from
May 1, 2023

Conversation

AlexWaygood
Copy link
Member

tomli is more actively maintained than toml, is more up to date with the TOML spec, and has been incorporated into the Python stdlib as tomllib in Python 3.11.

I think this is worth doing anyway, but this also helps with #282, as tomli is a py.typed package that ships with inline type annotations.

Comment on lines -7 to -18
packaging==21.3
packaging==23.1
# via -r requirements.in
psutil==5.9.0
psutil==5.9.5
# via
# -r requirements.in
# pyperf
pyparsing==3.0.8
# via packaging
pyperf==2.6.0
# via -r requirements.in
toml==0.10.2
# via -r requirements.in
Copy link
Member Author

@AlexWaygood AlexWaygood Apr 30, 2023

Choose a reason for hiding this comment

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

I ran pip-compile to regenerate this file, and it also bumped packaging and psutil. Not sure if that's desirable or if I did something wrong there! Not had much experience using pip-compile.

@AlexWaygood AlexWaygood requested a review from corona10 May 1, 2023 08:06
Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

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

lgtm!

@AlexWaygood AlexWaygood merged commit 0e745d2 into python:main May 1, 2023
11 checks passed
@AlexWaygood AlexWaygood deleted the tomli branch May 1, 2023 13:38
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.

None yet

2 participants