Skip to content
This repository was archived by the owner on Jan 7, 2022. It is now read-only.

[WIP] Feature/more eslint #48#50

Merged
jonshaffer merged 15 commits intomasterfrom
feature/more-eslint
Jan 9, 2017
Merged

[WIP] Feature/more eslint #48#50
jonshaffer merged 15 commits intomasterfrom
feature/more-eslint

Conversation

@jonshaffer
Copy link
Copy Markdown
Owner

@jonshaffer jonshaffer commented Jan 6, 2017

Adding eslintrc extending from jasmine, protractor, angular plugins and airbnb config.

Also adding stylelint to both codeclimate and locally.

Added changes for #48 #49

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 6, 2017

Current coverage is 19.89% (diff: 22.14%)

Merging #50 into master will decrease coverage by 0.18%

@@             master        #50   diff @@
==========================================
  Files            47         45     -2   
  Lines          1300       1322    +22   
  Methods         171        172     +1   
  Messages          0          0          
  Branches        303        307     +4   
==========================================
+ Hits            261        263     +2   
- Misses         1039       1059    +20   
  Partials          0          0          

Powered by Codecov. Last update f5f3115...e6b22be

@jonshaffer jonshaffer added this to the 0.8.0 milestone Jan 9, 2017
@jonshaffer jonshaffer changed the title Feature/more eslint #48 [WIP] Feature/more eslint #48 Jan 9, 2017
@jonshaffer
Copy link
Copy Markdown
Owner Author

jonshaffer commented Jan 9, 2017

Causing issues with material.css reference in demos, waiting on fixes for merge completion.

EDIT: somehow material.less had icon.less content, icon.less had datatable.less content. issue is fixed.

@jonshaffer jonshaffer merged commit 8483071 into master Jan 9, 2017
@jonshaffer jonshaffer deleted the feature/more-eslint branch January 9, 2017 22:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants