Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

build: using external rollup-plugin-smart-asset #7

Closed
wants to merge 116 commits into from
Closed

build: using external rollup-plugin-smart-asset #7

wants to merge 116 commits into from

Conversation

Tomyail
Copy link

@Tomyail Tomyail commented Sep 19, 2020

I found rollup-plugin-smart-asse's keepImport not working when rollup version < 2.0. this will cause all image build by microbundle become:

export default "public_path_to_asset"

not

const myAsset = require("relative_path_to_asset_from_bundle")

thus webpack's file-loader not working for bundle image.
I made a change of rollup-plugin-smart-asset to keep it work(inspired by this).
And i found microbundle is already try update rollup to 2.0. so this my be a temporary fix for microbundle. so i don't make a pr for rollup-plugin-smart-asse

ForsakenHarmony and others added 30 commits January 8, 2020 05:59
This option was enabled by default at some point in the node-resolve plugin, which should already have fixed developit#303 and developit#288. It still prints a warning though, which is illogical when we're compiling for a Node target. This silences that warning.
katywings and others added 29 commits July 4, 2020 15:33
Co-authored-by: Ward Peeters <ward@coding-tech.com>
See developit#673 

It's not possible to actually specify where to place the declarations file. This PR attempts to clarify the docs.
Co-authored-by: Leah <me@hrmny.sh>
Co-authored-by: Leah <github.leah@hrmny.sh>
…it#689)

unless I am mistaken the default export name for the module is in camelCase not snake
Co-authored-by: Ward Peeters <ward@coding-tech.com>
* Support global extend via amdName

Using a prefixed name like `{ "amdName": "global.foo" }` will trigger Rollup's `output.extend` option. This addresses the use-case from developit#709 without adding another CLI flag.

* Create chilled-paws-confess.md

* Update chilled-paws-confess.md
There is a small typo in test/index.test.js.

Should read `really` rather than `realy`.
Co-authored-by: Leah <github.leah@hrmny.sh>
Co-authored-by: Leah <github.leah@hrmny.sh>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Tomyail Tomyail closed this by deleting the head repository Feb 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.