-
Notifications
You must be signed in to change notification settings - Fork 154
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
Shims are not available in NPM package #791
Comments
I've shipped v1.3.1 to npm, do you want to try that? https://github.com/filerjs/filer/releases/tag/v1.3.1 |
Hello @humphd, As far as I can see, the shims are still missing in the NPM library v1.3.1. |
cc @bcheidemann |
@humphd the "shims" directory in the root of the project needs to be released to NPM. Could you advise on what changes need to be made to facilitate this? Happy to put up a PR. |
@thomas-jakemeyn this is the workaround until the files are released to NPM:
// src/filer/shims/fs.js
// replace
const { FileSystem } = require('../src/index');
// with
const { FileSystem } = require('filer');
// src/filer/shims/path.js
// replace
const { path } = require('../src/index');
// with
const { path } = require('filer');
// src/filer/shims/providers/default.js
// replace
const { Default } = require('../../src/providers/index');
// with
const { Default } = require('filer/src/providers/index');
// src/filer/shims/providers/indexeddb.js
// replace
const IndexedDB = require('../../src/providers/indexeddb');
// with
const IndexedDB = require('filer/src/providers/indexeddb');
// src/filer/shims/providers/memory.js
// replace
const Memory = require('../../src/providers/memory');
// with
const Memory = require('filer/src/providers/memory');
// webpack.config.js
module.exports = {
plugins: [
new filer.FilerWebpackPlugin({
shimsDir: '<rootDir>/src/filer/shims`,
}),
],
} This should work but I haven't tested it and I appreciate it's a lot of effort to go to. Hopefully it will be possible to release a new version of filer which includes the |
@bcheidemann ah, I see. We need to update https://github.com/filerjs/filer/blob/master/package.json#L77-L81 and potentially the |
Should be fixed by v1.4.1 |
Confirmed, shims are now present :) |
Context
I am trying to add
FilerWebpackPlugin
to an existing Webpack configuration.Problem
All imports of
fs
are properly replaced bynode_modules/filer/shims/fs.js
. However, that file cannot be found.Analysis
The shims are not made available in the NPM package.
The text was updated successfully, but these errors were encountered: