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.

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

Error while running npm run build #5

Closed
cioccarellia opened this issue Feb 6, 2017 · 4 comments
Closed

Error while running npm run build #5

cioccarellia opened this issue Feb 6, 2017 · 4 comments

Comments

@cioccarellia
Copy link
Member

I've encountered an error while executing npm run build


> app@1.0.0 build /Users/andrea/Desktop/client-master/app
> node build/build.js

Hash: 0ff2198b1357660ce539
Version: webpack 1.14.0
Time: 10059ms
                                                  Asset       Size  Chunks             Chunk Names
             static/js/manifest.5b6e18552e57cd348d16.js  833 bytes       0  [emitted]  manifest
                  static/js/app.9460d91753ff261d3732.js    7.66 kB    1, 0  [emitted]  app
               static/js/vendor.c8e7269ee28b9e50b0b5.js     111 kB    2, 0  [emitted]  vendor
    static/css/app.43051c1195135d9b56aa9b87e35d296b.css  636 bytes    1, 0  [emitted]  app
         static/js/manifest.5b6e18552e57cd348d16.js.map    8.87 kB       0  [emitted]  manifest
              static/js/app.9460d91753ff261d3732.js.map    51.6 kB    1, 0  [emitted]  app
static/css/app.43051c1195135d9b56aa9b87e35d296b.css.map    1.61 kB    1, 0  [emitted]  app
           static/js/vendor.c8e7269ee28b9e50b0b5.js.map     893 kB    2, 0  [emitted]  vendor
                                             index.html  945 bytes          [emitted]  

ERROR in ./src/pages/Settings.vue

  ✘  http://eslint.org/docs/rules/no-unused-vars  'save' is defined but never used  
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:31:10
  function save() {
            ^

  ✘  http://eslint.org/docs/rules/no-undef        'NativeStorage' is not defined    
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:32:3
    NativeStorage.setItem("backend_ip", ip);
     ^

  ✘  http://eslint.org/docs/rules/quotes          Strings must use singlequote      
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:32:25
    NativeStorage.setItem("backend_ip", ip);
                           ^

  ✘  http://eslint.org/docs/rules/no-undef        'ip' is not defined               
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:32:39
    NativeStorage.setItem("backend_ip", ip);
                                         ^

  ✘  http://eslint.org/docs/rules/no-undef        'NativeStorage' is not defined    
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:33:3
    NativeStorage.setItem("backend_user", ip);
     ^

  ✘  http://eslint.org/docs/rules/quotes          Strings must use singlequote      
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:33:25
    NativeStorage.setItem("backend_user", ip);
                           ^

  ✘  http://eslint.org/docs/rules/no-undef        'ip' is not defined               
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:33:41
    NativeStorage.setItem("backend_user", ip);
                                           ^

  ✘  http://eslint.org/docs/rules/no-undef        'NativeStorage' is not defined    
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:34:3
    NativeStorage.setItem("backend_password", ip);
     ^

  ✘  http://eslint.org/docs/rules/quotes          Strings must use singlequote      
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:34:25
    NativeStorage.setItem("backend_password", ip);
                           ^

  ✘  http://eslint.org/docs/rules/no-undef        'ip' is not defined               
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:34:45
    NativeStorage.setItem("backend_password", ip);
                                               ^

  ✘  http://eslint.org/docs/rules/no-undef        'Vue' is not defined              
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:35:3
    Vue.router.push("/");
     ^

  ✘  http://eslint.org/docs/rules/quotes          Strings must use singlequote      
  /Users/andrea/Desktop/client-master/app/src/pages/Settings.vue:35:19
    Vue.router.push("/");
                     ^


✘ 12 problems (12 errors, 0 warnings)


Errors:
  7  http://eslint.org/docs/rules/no-undef
  4  http://eslint.org/docs/rules/quotes
  1  http://eslint.org/docs/rules/no-unused-vars

ERROR in ./src/services/NetworkService.js

  ✘  https://google.com/#q=import%2Fnewline-after-import  Expected empty line after import statement not followed by another import  
  /Users/andrea/Desktop/client-master/app/src/services/NetworkService.js:2:1
  import axios from 'axios';
   ^

  ✘  http://eslint.org/docs/rules/max-len                 Line 5 exceeds the maximum line length of 100                              
  /Users/andrea/Desktop/client-master/app/src/services/NetworkService.js:5:1
      { ip_addr: '127.0.0.1', hw_addr: 'aa:bb:cc:dd', first_seen: new Date(), last_seen: new Date(), name: 'localhost' },
   ^

  ✘  http://eslint.org/docs/rules/max-len                 Line 6 exceeds the maximum line length of 100                              
  /Users/andrea/Desktop/client-master/app/src/services/NetworkService.js:6:1
      { ip_addr: '192.168.1.1', hw_addr: 'aa:bb:cc:de', first_seen: new Date(), last_seen: new Date(), name: 'router' },
   ^

  ✘  http://eslint.org/docs/rules/arrow-body-style        Unexpected block statement surrounding arrow body                          
  /Users/andrea/Desktop/client-master/app/src/services/NetworkService.js:8:19
    getHosts: () => {
                     ^

  ✘  http://eslint.org/docs/rules/no-trailing-spaces      Trailing spaces not allowed                                                
  /Users/andrea/Desktop/client-master/app/src/services/NetworkService.js:10:5
    }, 
       ^


✘ 5 problems (5 errors, 0 warnings)


Errors:
  2  http://eslint.org/docs/rules/max-len
  1  http://eslint.org/docs/rules/no-trailing-spaces
  1  http://eslint.org/docs/rules/arrow-body-style
  1  https://google.com/#q=import%2Fnewline-after-import

This is a new error because in the previous version it never happened

@Rubenoo
Copy link

Rubenoo commented Feb 6, 2017

Have you run " npm install " in the client/app before you did npm run build?

@cioccarellia
Copy link
Member Author

Yes, still getting the same error

@Silur
Copy link
Member

Silur commented Feb 7, 2017

Indeed, the code is poorly linted I hoped the CI would prevent this situation, will fix in a moment

@Silur
Copy link
Member

Silur commented Feb 7, 2017

fixed in 8ced4f8, thanks for the contribution

@Silur Silur closed this as completed Feb 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants