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

Belugas needs a git-init-ish directory, otherwise it crashes. #14

Open
thelastinuit opened this issue Mar 14, 2017 · 4 comments
Open

Belugas needs a git-init-ish directory, otherwise it crashes. #14

thelastinuit opened this issue Mar 14, 2017 · 4 comments

Comments

@thelastinuit
Copy link
Contributor

I guess this is all right but we need to be very specific on the documentation.

@mayra-cabrera
Copy link
Member

This is weird.. why it crashes?

@thelastinuit
Copy link
Contributor Author

Linguist has this class called Rugged something, it looks for git like structure (/.git/gitshit-directories) and get a lot of info apparently.
How did I notice it? Billy, passed me over one of his projects from his past job. It was a WAR file (like a JAR file but the W indicates that is a web development java project). So I run Whales on it, right?
It crashes. After I noticed that, I git-inited, git-added, git-committed the shit out of it and then it works.

@mayra-cabrera
Copy link
Member

Funny.. I understand that most of repos should be on git/github ... But what about if we make this extracting information from git step optional? 🤔 Thoughts? @kurenn @vovimayhem

@vovimayhem
Copy link
Member

vovimayhem commented Mar 14, 2017

This is an IcaliaLabs/belugas-linguist Issue. Actually, github/linguist is the one requiring it.

I have to say that it bothers me a bit, but the plan - at least Whales-wise - has always been to analyze repos.

I do see how I would want the requirement to not exist on belugas side, but considered it a minor priority.

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

3 participants