Create test.js #1

Open
wants to merge 53 commits into
from

Projects

None yet

2 participants

@nl5887
Contributor
nl5887 commented Jan 26, 2015

No description provided.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,17 @@
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+
+ doSomethingElse();
+ }
+
+// test for deleting old comments.
+
+
+ return a == null;
@linthubot
linthubot Feb 4, 2015 Owner

jshint.W116: Expected '===' and instead saw '=='.

@linthubot
linthubot Feb 4, 2015 Owner

jshint.W116: Expected '===' and instead saw '=='.

@linthubot
linthubot Feb 4, 2015 Owner

jshint.W116: Expected '===' and instead saw '=='.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,17 @@
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+
+ doSomethingElse();
+ }
+
+// test for deleting old comments.
+
+
@linthubot
linthubot Feb 4, 2015 Owner

jshint.W116: Expected '===' and instead saw '=='.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,17 @@
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+
@linthubot
linthubot Feb 4, 2015 Owner

jshint.W117: 'doSomethingElse' is not defined.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,17 @@
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+
+ doSomethingElse();
@linthubot
linthubot Feb 4, 2015 Owner

jshint.W117: 'doSomethingElse' is not defined.

@linthubot
linthubot Feb 4, 2015 Owner

jshint.W117: 'doSomethingElse' is not defined.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,16 @@
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+ doSomethingElse();
+ }
+
+// test for deleting old comments.
@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot
linthubot Feb 4, 2015 Owner

Trailing spaces not allowed. (no-trailing-spaces)

@linthubot linthubot commented on an outdated diff Feb 4, 2015
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+ doSomethingElse();
+ }
+
+// test for deleting old comments.
+// test to show pull request comments
+// eslint test9
+ return a == null;
+}
+
@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot
linthubot Feb 4, 2015 Owner

Unexpected blank line at end of file. (eol-last)

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,16 @@
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+ doSomethingElse();
+ }
+
+// test for deleting old comments.
@linthubot
linthubot Feb 4, 2015 Owner

****: Trailing spaces not allowed. (no-trailing-spaces)

@linthubot linthubot commented on an outdated diff Feb 4, 2015
+function main(a, b) {
+ switch (cond) {
+ case "one":
+ doSomething(); // JSHint will warn about missing 'break' here.
+ case "three":
+ doSomethingElse();
+ case "two":
+ doSomethingElse();
+ }
+
+// test for deleting old comments.
+// test to show pull request comments 3
+// eslint test9
+ return a == null;
+}
+
@linthubot
linthubot Feb 4, 2015 Owner

****: Unexpected blank line at end of file. (eol-last)

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,5 @@
+def badName
+ if something
@linthubot
linthubot Feb 4, 2015 Owner

Style/IndentationWidth: Use 2 (not 18) spaces for indentation.
Style/GuardClause: Use a guard clause instead of wrapping the code inside a conditional expression.
Style/IfUnlessModifier: Favor modifier if usage when having a single-line body. Another good alternative is the usage of control flow &&/||.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,5 @@
+def badName
+ if something
+ test
@linthubot
linthubot Feb 4, 2015 Owner

Style/IndentationWidth: Use 2 (not 4) spaces for indentation.

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,5 @@
+def badName
+ if something
+ test
+ end
@linthubot
linthubot Feb 4, 2015 Owner

Lint/EndAlignment: end at 4, 26 is not aligned with if at 2, 18

@linthubot linthubot commented on an outdated diff Feb 4, 2015
@@ -0,0 +1,5 @@
+def badName
+ if something
+ test
+ end
+ end
@linthubot
linthubot Feb 4, 2015 Owner

Lint/DefEndAlignment: end at 5, 26 is not aligned with def at 1, 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment