You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is a bug in src/plugin.ts:readManifest()
If both oclif.manifest.json and .oclif.manifest.json are missing, the exception is caught and mistakenly silenced.
The nesting level of the this.warn line might be wrong. No warning is issued if the .oclif.manifest.json is missing too.
Possible fix? if (!dotfile) return readManifest(true) else this.warn(error, 'readManifest').
} catch (error) {
if (error.code === 'ENOENT') { // both exceptions are caught here
if (!dotfile) return readManifest(true) // PROBLEM IS HERE
} else {
this.warn(error, 'readManifest')
}
}
There is a bug in
src/plugin.ts:readManifest()If both
oclif.manifest.jsonand.oclif.manifest.jsonare missing, the exception is caught and mistakenly silenced.The nesting level of the
this.warnline might be wrong. No warning is issued if the.oclif.manifest.jsonis missing too.Possible fix?
if (!dotfile) return readManifest(true) else this.warn(error, 'readManifest').