Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is where the world builds software
Millions of developers and companies build, ship, and maintain their software on GitHub — the largest and most advanced development platform in the world.
Fix Lints #230
Fix Lints #230
Comments
|
@bryanjos There are only ~750 of them or so :D I'm happy to start working on some of this for you, I need some real world Elixir experience. Do you have any special instructions (such as using --strict flag) or any specific way you want the PRs to be structured? If not, I'll probably opt to do a file at a time to minimize the scope of changes in each PR. |
|
@jodylecompte thanks! Without any flags is fine I think. I would ignore the |
|
@bryanjos I've had a chance to play around and get a little more familiar with the code base enough to go ahead and get started. My only question (so far) is regarding white space around parenthesis and curly braces. A large number of the errors Credo is registering are from such instances, but I wanted to check with you before removing them as they seem to be more by intent than by accident. For example, having a space behind a lodash/throwaway or a space between a curly brace and an atom. Would you rather these inconsistencies be removed, or Credo configured to ignore them? |
|
@jodylecompte I've always had spaces in curly brackets on purpose. That should probably be configured to be ignored. |
|
Hello! Neat project! I was picking through the lint for the JS-end tests and I noticed a lot of them had "unexpected console statement". Seeing as how that lint rule is aimed at browsers, and most of the JavaScript that it's warning on is testing the ElixirScript and outputting legible data for our purposes, would you like that warning disabled in |
|
Hi and yes that would be nice! |


There are a lot of them!
A good list to look at is here:
https://ebertapp.io/github/elixirscript/elixirscript
The "Quick Fixes" link shows the current list of lint issues