-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added RuboCop static code analyzer #12
base: master
Are you sure you want to change the base?
Conversation
Fix the RuboCop test and get the whole thing to work locally (also post some evidence that it works fine like screenshots or some output that RuboCop gives) then put a comment that it's done, I'll have a look then. Ignore the ruby-head test, it's failing in the repo too. |
@Uditgulati I think RuboCop linter is failing because GitHub token is not set for Ruby-Sparse repository. |
Why few checks failed ? Can you please have a look |
@Shekharrajak |
Do we have open issue for this ? If not, then please open one so that we can track it and someone pick this task. |
@Shekharrajak I have opened an issue #13 on |
Fixes: #4