-
Notifications
You must be signed in to change notification settings - Fork 926
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
fix(bit-ignore): support "require" syntax #6980
Conversation
@davidfirst I've tried this out and it seems to work except for an error being logged in the file where I'm using
This didn't happen prior to this patch. |
@evelant , any chance to get the content of this file so I could try reproducing it? |
@davidfirst the issue is here Lines 50 to 56 in 0b64e32
const addImportSpecifier = (dependency, importSpecifier) => {
if (dependencies[dependency]) {
if (dependencies[dependency].importSpecifiers) {
dependencies[dependency].importSpecifiers.push(importSpecifier);
} else {
dependencies[dependency].importSpecifiers = [importSpecifier];
}
}
}; |
See #6980 (comment) for more context.
Thanks. Fixed in #6988. |
No description provided.