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.

Sign up for GitHub

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

Jump to bottom

Test Hound #18

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

Test Hound #18

wants to merge 1 commit into from

Conversation

gylaz
Copy link
Owner

@gylaz gylaz commented Sep 8, 2018

No description provided.

hound[bot]
hound bot reviewed Sep 8, 2018
View reviewed changes
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

.eslintrc: 
.eslintrc:
	Configuration for rule "react/no-deprecated" is invalid:
	Value [{"react":"0.14.6"}] should NOT have more than 0 items.

Error: .eslintrc:

at validateRuleOptions (/app/node_modules/eslint/lib/config/config-validator.js:119:19)

at Object.keys.forEach.id (/app/node_modules/eslint/lib/config/config-validator.js:162:9)

at Array.forEach ()

at validateRules (/app/node_modules/eslint/lib/config/config-validator.js:161:30)

at Object.validate (/app/node_modules/eslint/lib/config/config-validator.js:239:5)

at loadFromDisk (/app/node_modules/eslint/lib/config/config-file.js:507:19)

at Object.load (/app/node_modules/eslint/lib/config/config-file.js:550:20)

at Config.getLocalConfigHierarchy (/app/node_modules/eslint/lib/config.js:228:44)

at Config.getConfigHierarchy (/app/node_modules/eslint/lib/config.js:180:43)

at Config.getConfigVector (/app/node_modules/eslint/lib/config.js:287:21)

test.py
@@ -0,0 +1,4 @@
x = 1
if x == 1:
print("x is 1.")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation is not a multiple of four

test.py
@@ -0,0 +1,4 @@
x = 1
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IndentationError: unexpected indent
unexpected indentation

test.ex
defmodule Test do

def print(foo,bar) do
IO.puts(foo, bar)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no trailing white-space at the end of a line.

test.ex
@@ -0,0 +1,6 @@
defmodule Test do
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modules should have a @moduledoc tag.

test.ex
@@ -0,0 +1,6 @@
defmodule Test do

def print(foo,bar) do
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space missing after comma

ruby_test.rb
LOGISTICS = [NAME, VERSION].freeze

def foo
puts 'bar'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

ruby_test.rb
# Test Hound
class TestHound
NAME = 'Wormbots'
VERSION = '0.0.1'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

TEST.md
@@ -0,0 +1,3 @@
* Hello
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect list-item indent: add 2 spaces

Repository owner deleted a comment from hound bot Sep 9, 2018
Repository owner deleted a comment from hound bot Sep 9, 2018
hound[bot]
hound bot reviewed Sep 9, 2018
View reviewed changes
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

.eslintrc: 
.eslintrc:
	Configuration for rule "react/no-deprecated" is invalid:
	Value [{"react":"0.14.6"}] should NOT have more than 0 items.

Error: .eslintrc:

at validateRuleOptions (/home/linters/node_modules/eslint/lib/config/config-validator.js:119:19)

at Object.keys.forEach.id (/home/linters/node_modules/eslint/lib/config/config-validator.js:162:9)

at Array.forEach ()

at validateRules (/home/linters/node_modules/eslint/lib/config/config-validator.js:161:30)

at Object.validate (/home/linters/node_modules/eslint/lib/config/config-validator.js:239:5)

at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:507:19)

at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20)

at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44)

at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)

at Config.getConfigVector (/home/linters/node_modules/eslint/lib/config.js:287:21)

Repository owner deleted a comment from hound bot Mar 23, 2019
Repository owner deleted a comment from hound bot Mar 23, 2019
hound[bot]
hound bot reviewed Mar 23, 2019
View reviewed changes
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

.eslintrc: 
.eslintrc:
	Configuration for rule "react/no-deprecated" is invalid:
	Value [{"react":"0.14.6"}] should NOT have more than 0 items.

Error: .eslintrc:

at validateRuleOptions (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:119:19)

at Object.keys.forEach.id (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:162:9)

at Array.forEach ()

at validateRules (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:161:30)

at Object.validate (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:239:5)

at loadFromDisk (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-file.js:516:19)

at Object.load (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-file.js:559:20)

at Config.getLocalConfigHierarchy (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:227:44)

at Config.getConfigHierarchy (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:179:43)

at Config.getConfigVector (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:286:21)

test.py
x = 1
if x == 1:
print("x is 1.")

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

blank line at end of file

ruby_test.rb

# Test Hound
class TestHound
NAME = 'Wormbots'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

hound[bot]
hound bot reviewed Apr 2, 2019
View reviewed changes
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

.eslintrc: 
.eslintrc:
	Configuration for rule "react/no-deprecated" is invalid:
	Value [{"react":"0.14.6"}] should NOT have more than 0 items.

Error: .eslintrc:

at validateRuleOptions (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:119:19)

at Object.keys.forEach.id (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:162:9)

at Array.forEach ()

at validateRules (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:161:30)

at Object.validate (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:239:5)

at loadFromDisk (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-file.js:516:19)

at Object.load (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-file.js:559:20)

at Config.getLocalConfigHierarchy (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:227:44)

at Config.getConfigHierarchy (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:179:43)

at Config.getConfigVector (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:286:21)

hound[bot]
hound bot reviewed Apr 2, 2019
View reviewed changes
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

.eslintrc: 
.eslintrc:
	Configuration for rule "react/no-deprecated" is invalid:
	Value [{"react":"0.14.6"}] should NOT have more than 0 items.

Error: .eslintrc:

at validateRuleOptions (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:119:19)

at Object.keys.forEach.id (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:162:9)

at Array.forEach ()

at validateRules (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:161:30)

at Object.validate (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-validator.js:239:5)

at loadFromDisk (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-file.js:516:19)

at Object.load (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config/config-file.js:559:20)

at Config.getLocalConfigHierarchy (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:227:44)

at Config.getConfigHierarchy (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:179:43)

at Config.getConfigVector (/home/linters/app/versions/eslint-4.18.2/node_modules/eslint/lib/config.js:286:21)

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

@hound hound[bot] hound[bot] left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gylaz