Skip to content
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 package and add separate filepicker entry point #1070

Merged
merged 4 commits into from
Oct 17, 2023
Merged

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Oct 17, 2023

  1. Combine vite and vitest config
  2. Remove non existing legacy entry point from package.json
  3. Do not export the filepicker Vue component in main entry point. Instead export it in a filepicker entrypoint, also load the component async. in the filepicker builder. This ensures that if a user does not use the filepicker, it gets treeshaken.

Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
…e.json

Also move mdi dependency to dependencies instead of devDependencies

Signed-off-by: Ferdinand Thiessen <[email protected]>
Instead export it in a `filepicker` entrypoint, also load the component async. in the filepicker builder.
This ensures that if a user does not use the filepicker, it gets treeshaken.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux added bug Something isn't working 3. to review labels Oct 17, 2023
@susnux susnux changed the title Refactor structure Fix package and add separate filepicker entry point Oct 17, 2023
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Merging #1070 (bc789de) into main (1e08381) will decrease coverage by 0.72%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1070      +/-   ##
==========================================
- Coverage   31.15%   30.44%   -0.72%     
==========================================
  Files          15       16       +1     
  Lines        1194     1222      +28     
  Branches       60       61       +1     
==========================================
  Hits          372      372              
- Misses        764      791      +27     
- Partials       58       59       +1     
Files Coverage Δ
lib/index.ts 0.00% <0.00%> (ø)
lib/filepicker.ts 0.00% <0.00%> (ø)
lib/filepicker-builder.ts 0.00% <0.00%> (ø)

@susnux susnux merged commit a29bb7f into main Oct 17, 2023
10 of 12 checks passed
@susnux susnux deleted the refactor-structure branch October 17, 2023 15:27
Comment on lines -15 to -17
"./dist/index.css": {
"import": "./dist/style.css",
"require": "./dist/style.css"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants