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

Add Clojure support #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sarcilav
Copy link

This includes comments recognition and basic metrics.

@adam-waldenberg
Copy link
Member

Hi @sarcilav. Looks interesting. We can add it as a default extension, but I would prefer that the support for the metrics is "complete". I really don't kow much about clojure, but looking at it seems the "(" character is also used within if statements and other things.

@sarcilav
Copy link
Author

sarcilav commented Dec 6, 2016

@adam-waldenberg true, the thing is that there is not the concept of if as an statement or reserve word, it is just another function

@adam-waldenberg
Copy link
Member

@sarcilav What about the macros covered in;

http://clojure.org/api/cheatsheet

Looks to me like the branch and loop sections under macros might be candidates to add to the regexp in gitinspector. Then again, I'm guessing these are just some macros defined by default and you could potentially do anything you want ?

Maybe the best course of action is to indeed use "(".

This includes comments recognition and basic metrics.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants