Skip to content

Commit 4aa018f

Browse files
Merge #1333
1333: Fix rollup importing wrong 'meilisearch' in auto-complete r=curquiza a=flevi29 # Pull Request ## Related issue Fixes #1330 ## PR checklist Please check if your PR fulfills the following requirements: - [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)? - [x] Have you read the contributing guidelines? - [x] Have you made sure that the title is accurate and descriptive of the changes? Thank you so much for contributing to Meilisearch! Co-authored-by: F. Levi <[email protected]>
2 parents 3de2804 + 7bc6b54 commit 4aa018f

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

.changeset/calm-snakes-dream.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@meilisearch/autocomplete-client": patch
3+
---
4+
5+
Fix rollup importing wrong "meilisearch"

packages/autocomplete-client/rollup.config.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,7 @@ const ROLLUP_OPTIONS = [
5151
},
5252
plugins: [
5353
...COMMON_PLUGINS,
54-
nodeResolve({
55-
mainFields: ['jsnext', 'browser', 'main'],
56-
preferBuiltins: true,
57-
browser: true,
58-
}),
54+
nodeResolve({ exportConditions: ['browser'] }),
5955
commonjs(),
6056
babel(),
6157
env === 'production' ? terser() : {}, // will minify the file in production mode

0 commit comments

Comments
 (0)