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 the entrypoint asn1ate available again #67

Closed
wants to merge 2 commits into from
Closed

Make the entrypoint asn1ate available again #67

wants to merge 2 commits into from

Conversation

rockwelln
Copy link
Contributor

In the current repository state, the asn1ate entrypoint exposed by setup.py fails with the error message:

$ asn1ate nrtrde.asn
Traceback (most recent call last):
  File "/note/.virtualenvs/sandbox/bin/asn1ate", line 11, in <module>
    load_entry_point('asn1ate', 'console_scripts', 'asn1ate')()
TypeError: main() missing 1 required positional argument: 'args'

Here is a little fix to do not break what was done for the unit tests and make the command running again :-).

@kimgr
Copy link
Owner

kimgr commented Nov 12, 2018

Thank you! Can you squash the two commits into one?

@kimgr
Copy link
Owner

kimgr commented Nov 21, 2018

I'll take charge of this, will try my best to preserve your name on the commit.

@rockwelln
Copy link
Contributor Author

Sorry, I didn't had the time to take care of this one before.
I'll have some time to look into it on Friday.

@kimgr
Copy link
Owner

kimgr commented Nov 21, 2018

Squashed and merged in 131411b. Thanks!

@kimgr kimgr closed this Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants