facebookarchive / atom-ide-ui

A collection of user interfaces for Atom IDE.
http://ide.atom.io
Other
934 stars 79 forks source link

Argument to [path.dirname|path.extname] must be a string #195

Closed michael-gracy closed 6 years ago

michael-gracy commented 6 years ago

Before Submitting

Description

Argument to path.dirname must be a string stack: Object.dirname - /Users/michael.gracy/Applications/Atom.app/Contents/Resources/app.asar/src/electron-shims.js:9:10 Object.lint - /Users/michael.gracy/.atom/packages/linter-shellcheck/lib/main.js:61:24 a.factory - /Users/michael.gracy/.atom/packages/atom-ide-ui/modules/atom-ide-ui/pkg/atom-ide-diagnostics/lib/services/LinterAdapter.js:263:42 a._callFactory - /Users/michael.gracy/.atom/packages/atom-ide-ui/node_modules/rxjs/bundles/Rx.min.js:107:23 a.tryDefer - /Users/michael.gracy/.atom/packages/atom-ide-ui/node_modules/rxjs/bundles/Rx.min.js:106:442 new a - /Users/michael.gracy/.atom/packages/atom-ide-ui/node_modules/rxjs/bundles/Rx.min.js:106:383

Argument to path.extname must be a string stack: Object.extname - /Users/michael.gracy/Applications/Atom.app/Contents/Resources/app.asar/src/electron-shims.js:20:10 Object.lint - /Users/michael.gracy/.atom/packages/linter-shellcheck/lib/main.js:55:28 a.factory - /Users/michael.gracy/.atom/packages/atom-ide-ui/modules/atom-ide-ui/pkg/atom-ide-diagnostics/lib/services/LinterAdapter.js:263:42 a._callFactory - /Users/michael.gracy/.atom/packages/atom-ide-ui/node_modules/rxjs/bundles/Rx.min.js:107:23 a.tryDefer - /Users/michael.gracy/.atom/packages/atom-ide-ui/node_modules/rxjs/bundles/Rx.min.js:106:442 new a - /Users/michael.gracy/.atom/packages/atom-ide-ui/node_modules/rxjs/bundles/Rx.min.js:106:383

Expected Behavior

No error message

Actual Behavior

Get above error message after auto-update of atom-ide-ui

Versions

Additional Details

wanderley commented 6 years ago

Can you provide more context about this issue? Is there a way to repro it?

From the stacktrace

Object.lint - /Users/michael.gracy/.atom/packages/linter-shellcheck/lib/main.js:61:24

It seems to be a problem related with linter-shellcheck. Can this issue be related with this package?

Arcanemagus commented 6 years ago

I'm guessing this is a duplicate of https://github.com/AtomLinter/linter-shellcheck/issues/107 and https://github.com/AtomLinter/linter-shellcheck/issues/108 which were fixed... but apparently never released.

I'll push that out right now 😉.