stricter lint rules #1162
Replies: 4 comments
-
@inlang/team |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Beta Was this translation helpful? Give feedback.
0 replies
-
I have come across every point and agree to take action. We should commit the code as clean as possible and without changes by running the current version. |
Beta Was this translation helpful? Give feedback.
0 replies
-
@openscript agrees too, I have spoken with him already about these issues |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I want to share a few issues with the current code base:
console.log
s in the code (see issue)console.log
are mosten used for debugging and often forgotten to remove. If there really is the case where we want to log something we can useconsole.info
for that.npm i
. It happens to often that I branch of from main, runnpm i
and have changes inpackage_lock.json
. We should enforce thatpackage_lock.json
must be up2date.Beta Was this translation helpful? Give feedback.
All reactions