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

Make this pip installable #3

Open
SylvainDe opened this issue Oct 28, 2014 · 13 comments
Open

Make this pip installable #3

SylvainDe opened this issue Oct 28, 2014 · 13 comments

Comments

@SylvainDe
Copy link
Owner

SylvainDe added a commit that referenced this issue Oct 29, 2014
SylvainDe added a commit that referenced this issue Oct 29, 2014
@SylvainDe
Copy link
Owner Author

$ python setup.py register -r pypi
running register
running check
Registering didyoumean to https://pypi.python.org/pypi
Server response (403): Package name conflicts with existing package 'DidYouMean'

I guess I'll have to find another name.

@SylvainDe
Copy link
Owner Author

Also, I forgot to put the init.py and I am not quite sure how to make the whole thing work.

@SylvainDe
Copy link
Owner Author

@sametmax
Copy link

sametmax commented Mar 6, 2017

What more do you need ? I can help you wish packaging, but appart for the name, you seem to be ok. Do you have another problem ?

@SylvainDe
Copy link
Owner Author

Hi @sametmax , quite an honor to see you here :)
I guess I'll need to try with another name. I'll keep you posted but I'm very new to this packaging thing.
If you have any suggestion regarding the name, you are more than welcome!

@sametmax
Copy link

sametmax commented Mar 6, 2017

:) It's a selfish visit. I'm gonna add DidYouMean to devpy.

DidYouMean could mean a spell checker or something from a search engine so adding a qualifier in the name wouldn't hurt anyway.

BetterErrorMessages, ErrorHints, ExceptionHints, ErrorTips, ExceptionTips, etc could be more self-explanatory names.

@SylvainDe
Copy link
Owner Author

@SylvainDe
Copy link
Owner Author

$ python setup.py register -r pypitest
Registering BetterErrorMessages to https://upload.pypi.org/legacy/
Server response (410): This API is no longer supported, instead simply upload the file.

:(

@SylvainDe
Copy link
Owner Author

I'm getting punished for procrastinating on this issue :-)

@SylvainDe
Copy link
Owner Author

It works!!!!!

$ virtualenv toto_env
$ source toto_env/bin/activate
$ pip install BetterErrorMessages
$ python
>>> import didyoumean
>>> didyoumean.didyoumean_api.didyoumean_enablehook()
>>> abcdef = 'toto'
>>> abcdf
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'abcdf' is not defined. Did you mean 'abcdef' (local)?

@SylvainDe
Copy link
Owner Author

SylvainDe commented Mar 6, 2017

@sametmax Now it is pretty obvious (from the example above) that the API is terrible. I reckon being able to do import didyoumean and didyoumean.enablehook would probably be MUCH better. As you'll probably be the first user, you get to decide if that would be fine for you before it's too late :-).
Also, I should be able to do so just by messing with the __init__.py file, right ?

@SylvainDe
Copy link
Owner Author

SylvainDe commented Mar 6, 2017

  • Make the API easier to use
  • Update the readme

@SylvainDe
Copy link
Owner Author

@sametmax Any idea regarding the question above ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants