Pewarta Proxy

📋 Server Request & Proxy Logs

Loading active proxy settings...
Developed by Khairul Adha (adhexa.id)
Mengambil log server...

⚙️ Pengaturan Proxy Target

Skip to content

Allow "-m json" instead of "-m json.tool" #122873

Closed
Closed
Allow "-m json" instead of "-m json.tool"#122873
Labels
stdlibStandard Library Python modules in the Lib/ directorytype-featureA feature request or enhancement

Description

@treyhunner
treyhunner
opened on Aug 9, 2024

Feature or enhancement

Proposal:

This is a feature proposal for allowing python3 -m json to work in addition to python -m json.tool and softly deprecating the use of python3 -m json.tool.

I made a branch to see what these changes might look like.

Here is a separate branch which issues a DeprecationWarning .
However, since json.tool will continue working that warning may cause more hassle for end users than is worthwhile.

I think the no-warning approach may be more sensible.

Has this already been discussed elsewhere?

I have already discussed this feature proposal on Discourse

Links to previous discussion of this feature:

Discussion in Ideas discuss thread

Linked PRs

Activity

  1. treyhunner
    added
    type-featureA feature request or enhancement
    on Aug 9, 2024
  2. hugovk
    added
    3.14bugs and security fixes
    on Aug 9, 2024

  3. ncoghlan commented on Aug 10, 2024

    @ncoghlan
    ncoghlan
    on Aug 10, 2024
    Contributor

    The "No warning" branch looks good to me (it would just need docs and News for me to approve a PR)

  4. Eclips4
    added
    stdlibStandard Library Python modules in the Lib/ directory
    on Aug 10, 2024

  5. Eclips4 commented on Aug 10, 2024

    @Eclips4
    Eclips4
    on Aug 10, 2024
    Member

    Hi @hugovk! Just a quick note: type-feature label doesn't require a version label: https://devguide.python.org/triage/labels/#type-labels

  6. Eclips4
    removed
    3.14bugs and security fixes
    on Aug 10, 2024

  7. hugovk commented on Aug 10, 2024

    @hugovk
    hugovk
    on Aug 10, 2024
    Member

    The "No warning" branch looks good to me (it would just need docs and News for me to approve a PR)

    Agreed.

    And change this in Lib/json/tool.py?

     def main():
-    prog = 'python -m json.tool'
+    prog = 'python -m json'

    @treyhunner Would you like to put together a PR?

  8. treyhunner commented on Aug 10, 2024

    @treyhunner
    treyhunner
    on Aug 10, 2024
    MemberAuthor

    I will put together a PR shortly. 👍

  9. rhettinger commented on Aug 10, 2024

    @rhettinger
    rhettinger
    on Aug 10, 2024
    Contributor

    +1 from me

  10. hugovk commented on Aug 11, 2024

    @hugovk
    hugovk
    on Aug 11, 2024
    Member

    PR is here: #122884

  11. AA-Turner
    added a commit that references this issue on Aug 13, 2024

    gh-122873: Allow "python -m json" to work (#122884)

    906b796
  12. hugovk
    closed this as completedon Aug 13, 2024
  13. blhsing
    added a commit that references this issue on Aug 22, 2024

    pythongh-122873: Allow "python -m json" to work (python#122884)

    8b27d24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    stdlibStandard Library Python modules in the Lib/ directorytype-featureA feature request or enhancement

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions