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

read-project-clj reading only first form #24

Open
mpenet opened this issue Nov 5, 2015 · 4 comments
Open

read-project-clj reading only first form #24

mpenet opened this issue Nov 5, 2015 · 4 comments

Comments

@mpenet
Copy link

mpenet commented Nov 5, 2015

https://github.com/hashobject/jarkeeper.com/blob/master/src/jarkeeper/core.clj#L60

nothing guarantees that the first form is the (project ...), there could be vars, macros and so on.

@podviaznikov
Copy link
Member

@mpenet agree with you. I think I never saw project.clj like that but your suggestion makes sense

@podviaznikov
Copy link
Member

@mpenet
Copy link
Author

mpenet commented Nov 5, 2015

:}

There are a few others out there

@podviaznikov
Copy link
Member

Hmm, I wasn't able to fully fix this problem. https://jarkeeper.com/mpenet/alia - drivers versions are not reported correctly.

@podviaznikov podviaznikov reopened this Nov 6, 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
Development

No branches or pull requests

2 participants