Skip to content
This repository was archived by the owner on Mar 10, 2020. It is now read-only.

chore: re-enable linting #324

Merged
merged 1 commit into from
Jul 5, 2018
Merged

chore: re-enable linting #324

merged 1 commit into from
Jul 5, 2018

Conversation

alanshaw
Copy link
Contributor

@alanshaw alanshaw commented Jul 5, 2018

When the JS code was moved into the js/ directory the aegir linting task was no longer considering them for linting. This is because the default globs do not include this path.

This PR adds custom lint globs to the aegir config and fixes linting errors.

depends on ipfs/aegir#240

License: MIT
Signed-off-by: Alan Shaw [email protected]

When the JS code was moved into the js/ directory the aegir linting task was no longer considering them for linting. This is because the default globs do not include this path.

This PR adds custom lint globs to the aegir config and fixes linting errors.

depends on ipfs/aegir#240

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
@ghost ghost assigned alanshaw Jul 5, 2018
@ghost ghost added the in progress label Jul 5, 2018
Copy link
Contributor

@travisperson travisperson left a comment

Choose a reason for hiding this comment

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

LGTM. I'm impressed that there were basically no issues 👍

@alanshaw
Copy link
Contributor Author

alanshaw commented Jul 5, 2018

I picked off most of them when I did the modularization PR 😉

@alanshaw alanshaw merged commit 87a8f96 into master Jul 5, 2018
@ghost ghost removed the in progress label Jul 5, 2018
@alanshaw alanshaw deleted the chore/re-enable-linting2 branch July 5, 2018 19:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants