Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

Add npm lint script and use it on Travis #137

Merged
merged 2 commits into from
Feb 22, 2017
Merged

Add npm lint script and use it on Travis #137

merged 2 commits into from
Feb 22, 2017

Conversation

rhoggSugarcrm
Copy link
Contributor

For now, this just defers to gulp lint, but we can change that when we get to #130.

@rhoggSugarcrm rhoggSugarcrm self-assigned this Feb 16, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 94.22% when pulling 42f4852 on rhoggSugarcrm:npm-run-lint into b5fd831 on sugarcrm:master.

jcsmorais
jcsmorais approved these changes Feb 21, 2017
View reviewed changes
Copy link
Contributor

@jcsmorais jcsmorais left a comment

Choose a reason for hiding this comment

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

:shipit:

@rhoggSugarcrm rhoggSugarcrm merged commit c7c2df0 into sugarcrm:master Feb 22, 2017
@rhoggSugarcrm rhoggSugarcrm deleted the npm-run-lint branch February 22, 2017 01:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jcsmorais jcsmorais jcsmorais approved these changes

Assignees

@rhoggSugarcrm rhoggSugarcrm

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@rhoggSugarcrm @coveralls @jcsmorais