Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

The input screen is unclear and may put invalid characters in class names. #61

Open
Poikilos opened this issue Jun 12, 2022 · 0 comments

Comments

@Poikilos
Copy link

Poikilos commented Jun 12, 2022

Expected behavior

  1. What answers become what is not totally clear. The on the first run of the cookiecutter, I put the project name in "full_name" then cancelled and went back and put my name. Maybe making the project name first would be more clear, and if there is a way to put a tip along with full_name say "author full name".
  2. Stripping out bad characters or mentioning the string is for "class names" would be helpful (or having a tip saying only spaces, letters and numbers and must start with letter), because the following answers produced unexpected results:
project_name [Spyder Boilerplate]: Context-Aware Translate           
...
project_pypi_name [context-aware-translate]: catranslate
project_package_name [context_aware_translate]: catranslate
...

Actual behavior

(The hyphen is used in class names, creating invalid syntax.)

widgets.py

class Context-AwareTranslateActions:
    ExampleAction = "example_action"


class Context-AwareTranslateToolBarSections:
    ExampleSection = "example_section"


class Context-AwareTranslateOptionsMenuSections:
    ExampleSection = "example_section"

confpage.py

class Context-AwareTranslateConfigPage(PluginConfigPage):

plugin.py

class Context-AwareTranslate(SpyderDockablePlugin):
...
WIDGET_CLASS = Context-AwareTranslateWidget
...
CONF_WIDGET_CLASS = Context-AwareTranslateConfigPage

setup.py

catranslate = catranslate.spyder.plugin:Context-AwareTranslate
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant