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

Jump to bottom

IDLE: revise zzdummy.py #76812

Closed
terryjreedy opened this issue Jan 23, 2018 · 5 comments
Closed

IDLE: revise zzdummy.py #76812

terryjreedy opened this issue Jan 23, 2018 · 5 comments
Assignees
@terryjreedy
Labels
3.8 only security fixes 3.9 only security fixes 3.10 only security fixes topic-IDLE type-bug An unexpected behavior, bug, or error

Comments

@terryjreedy
Copy link
Member

BPO 32631
Nosy @terryjreedy
PRs
  • bpo-32631: IDLE: Enable zzdummy extension module #14491
  • [3.9] bpo-32631: IDLE: Enable zzdummy example extension module (GH-14… #24114
  • [3.8] bpo-32631: IDLE: Enable zzdummy example extension module (GH-14491) #24115
    		•

    Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/terryjreedy'
closed_at = <Date 2021-01-05.09:18:33.266>
created_at = <Date 2018-01-23.06:57:24.596>
labels = ['expert-IDLE', 'type-bug', '3.8', '3.9', '3.10']
title = 'IDLE: revise zzdummy.py'
updated_at = <Date 2021-01-05.09:18:33.266>
user = 'https://github.com/terryjreedy'

    bugs.python.org fields:

    activity = <Date 2021-01-05.09:18:33.266>
actor = 'terry.reedy'
assignee = 'terry.reedy'
closed = True
closed_date = <Date 2021-01-05.09:18:33.266>
closer = 'terry.reedy'
components = ['IDLE']
creation = <Date 2018-01-23.06:57:24.596>
creator = 'terry.reedy'
dependencies = []
files = []
hgrepos = []
issue_num = 32631
keywords = ['patch']
message_count = 5.0
messages = ['310473', '384372', '384376', '384377', '384378']
nosy_count = 1.0
nosy_names = ['terry.reedy']
pr_nums = ['14491', '24114', '24115']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue32631'
versions = ['Python 3.8', 'Python 3.9', 'Python 3.10']
    @terryjreedy
    Copy link
    Member Author

    zzdummy.py is needed for properly testing the extension mechanism. It is intended to provide an example extension for users to follow. When completed, enabled, and invoked, it will prefix every line of a window with some text, or remove the prefix from every line. It would be even better to make it respect selections, as other format options do.

    Needed: more comments, including what it does when enabled; docstring for z_in_event; uncomment z_in_event; code for z_out_event.

    Should the extension have a different name? ZzTestExample? ZzIgnoreThis? Changing an extension name would normally be a bad idea, as it would disable any customizations. But I think we could do it for this one if desired.

    The default insertion text could be more informative: for instance, 'ignore this' or 'see-doc' or #ignore#. The last would make it a configurable comment/uncomment. Normally, changing a default is a no-no, as it can mess up user configurations. But since the current text is inoperative, this should not matter here.

    The menu defs are commented out because enabling the extension currently has no effect, so the menu entries would be useless.

    ***There should be help text for the extension tab explaining the presence and operation of the extension. (This should have been done already.)

    Extension tests, last revised in bpo-31480, will likely need more changes.

    @terryjreedy terryjreedy added the 3.7 (EOL) end of life label Jan 23, 2018
    @terryjreedy terryjreedy self-assigned this Jan 23, 2018
    @terryjreedy terryjreedy added topic-IDLE type-bug An unexpected behavior, bug, or error labels Jan 23, 2018
    @csabella csabella added 3.9 only security fixes 3.10 only security fixes and removed 3.7 (EOL) end of life labels Jun 13, 2020
    @terryjreedy
    Copy link
    Member Author

    New changeset e40e2a2 by Cheryl Sabella in branch 'master':
    bpo-32631: IDLE: Enable zzdummy example extension module (GH-14491)
    e40e2a2

    @terryjreedy
    Copy link
    Member Author

    New changeset d82392f by Terry Jan Reedy in branch '3.9':
    [3.9] bpo-32631: IDLE: Enable zzdummy example extension module (GH-14491)
    d82392f

    @terryjreedy
    Copy link
    Member Author

    https://stackoverflow.com/questions/65567057/how-can-i-get-started-developing-extensions-for-python-idle prompted me to finish this enough to merge it.

    My intention was that in the absence of a selection, all lines should be prefixed, not just the current one. My impression is that some other formatters do this, but maybe not. In any case, it does not matter exactly what this dummy example does, as long as it works as documented.

    @terryjreedy terryjreedy added the 3.8 only security fixes label Jan 5, 2021
    @terryjreedy
    Copy link
    Member Author

    New changeset a087a97 by Terry Jan Reedy in branch '3.8':
    [3.8] bpo-32631: IDLE: Enable zzdummy example extension module (GH-14491)
    a087a97

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Assignees

    @terryjreedy terryjreedy

    Labels
    3.8 only security fixes 3.9 only security fixes 3.10 only security fixes topic-IDLE type-bug An unexpected behavior, bug, or error
    Projects
    None yet
    Milestone
    No milestone
    Development

    No branches or pull requests
    2 participants
    @terryjreedy @csabella