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

generator.rb - dead code? #135

Closed
voxik opened this issue Jul 17, 2013 · 2 comments · Fixed by #572
Closed

generator.rb - dead code? #135

voxik opened this issue Jul 17, 2013 · 2 comments · Fixed by #572

Comments

@voxik
Copy link
Contributor

voxik commented Jul 17, 2013

https://github.com/Pajk/apipie-rails/blob/master/lib/apipie/client/generator.rb seems to be dead code. It references another files, which are not available in repository: https://github.com/Pajk/apipie-rails/blob/master/lib/apipie/client/generator.rb#L8

Also note that the generator.rb uses /bin/env shebang, which might be issue for Fedora, that's where came my interest in this file ;)

[1] https://lists.fedoraproject.org/pipermail/devel/2013-July/185797.html

@voxik
Copy link
Contributor Author

voxik commented Nov 14, 2017

ping

Any update? I think I am going to drop the file from Fedora for the moment, since I was asked if I can change the shebang ...

@iNecas
Copy link
Member

iNecas commented Nov 14, 2017

Yes, this is a dead code we can remove

voxik added a commit to voxik/apipie-rails that referenced this issue Nov 16, 2017
apipie/client/generator.rb seems to be dead code. Moreover, it references
another files, such as apipie/client/base, which are neither available in
repository (resolves Apipie#135).
iNecas pushed a commit that referenced this issue Nov 17, 2017
apipie/client/generator.rb seems to be dead code. Moreover, it references
another files, such as apipie/client/base, which are neither available in
repository (resolves #135).
yoav-p6 pushed a commit to parallel-6/apipie-rails that referenced this issue Nov 6, 2020
apipie/client/generator.rb seems to be dead code. Moreover, it references
another files, such as apipie/client/base, which are neither available in
repository (resolves Apipie#135).
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 a pull request may close this issue.

2 participants