Skip to content

gh-138019: better help messge for json.tool same as ast #138037

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

Merged
merged 4 commits into from
Aug 22, 2025

Conversation

yihong0618
Copy link
Contributor

@yihong0618 yihong0618 commented Aug 22, 2025

As issue comment, in #138019
maybe we can provide better help message for infile the same as ast and tokenize
cc @picnixz @StanFromIreland

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@picnixz
Copy link
Member

picnixz commented Aug 22, 2025

That would indeed be helpful. Is it the same as for ast and tokenize?

@yihong0618
Copy link
Contributor Author

yihong0618 commented Aug 22, 2025

That would indeed be helpful. Is it the same as for ast and tokenize?

yes both ast and tokenize have this message

image

@StanFromIreland
Copy link
Member

Please do not use the Update Branch button unless necessary (e.g. fixing conflicts, jogging the CI, or very old PRs) as it uses valuable resources. For more information see the devguide.

@picnixz picnixz enabled auto-merge (squash) August 22, 2025 08:20
@picnixz
Copy link
Member

picnixz commented Aug 22, 2025

Thank you! (please don't commit anything now)

@yihong0618
Copy link
Contributor Author

Please do not use the Update Branch button unless necessary (e.g. fixing conflicts, jogging the CI, or very old PRs) as it uses valuable resources. For more information see the devguide.

learned that, thank you

@picnixz picnixz merged commit 531fc3a into python:main Aug 22, 2025
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants