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

master tries to import parse_options, which no longer exists #92

Closed
lvh opened this issue Jan 8, 2015 · 2 comments
Closed

master tries to import parse_options, which no longer exists #92

lvh opened this issue Jan 8, 2015 · 2 comments

Comments

@lvh
Copy link

@lvh lvh commented Jan 8, 2015

pip install git+https://github.com/GreenSteam/pep257.git@8b331a850a45c57c43a762ff8133bcdea0bb3956#egg=pep257                                                                           09:21:05
Collecting git+https://github.com/GreenSteam/pep257.git@8b331a850a45c57c43a762ff8133bcdea0bb3956
  Cloning https://github.com/GreenSteam/pep257.git (to 8b331a850a45c57c43a762ff8133bcdea0bb3956) to /var/folders/2w/kffqc_pj6m38rgrxlhk8z6nr0000gn/T/pip-gRke1t-build
  Could not find a tag or branch '8b331a850a45c57c43a762ff8133bcdea0bb3956', assuming commit.
Installing collected packages: pep257
  Running setup.py install for pep257
    changing mode of build/scripts-2.7/pep257 from 644 to 755
    changing mode of /Users/lvh/.pyenv/versions/otter/bin/pep257 to 755
Successfully installed pep257-0.4.0
~/C/r/otter: pep257                                                                                                                                                                                 09:21:10
Traceback (most recent call last):
  File "/Users/lvh/.pyenv/versions/otter/bin/pep257", line 3, in <module>
    from pep257 import main, parse_options
ImportError: cannot import name parse_options

#75 might fix this accidentally (by replacing the current option parser). Cc @Nurdok?

@Nurdok
Copy link
Member

@Nurdok Nurdok commented Jan 9, 2015

It looks like I forgot to change this file while working on the config file support PR. Although the main failure here is that there are no tests for the script file. I will attempt to fix this today.

@Nurdok
Copy link
Member

@Nurdok Nurdok commented Jan 9, 2015

Solved by #93.

@Nurdok Nurdok closed this Jan 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants