Niklas Mischkulnig

Results 290 comments of Niklas Mischkulnig

Usually, Postcss gives back a list of dependencies to watch for changes https://github.com/parcel-bundler/parcel/blob/ce58970eb70cc8c8dcf9cd6e6506761021e0cc7e/packages/transformers/postcss/src/PostCSSTransformer.js#L217-L219 but if the preceding call thrown an exception, then that never happens https://github.com/parcel-bundler/parcel/blob/ce58970eb70cc8c8dcf9cd6e6506761021e0cc7e/packages/transformers/postcss/src/PostCSSTransformer.js#L208-L211 This is similar to...

> Presumably the issue was introduced in some commit between 2023-03-25 and 2023-03-27. https://github.com/parcel-bundler/parcel/pull/8723 fits the merge date and also your description of the bug

So yet another breaking change just like https://github.com/parcel-bundler/parcel/issues/9405...

This should be working. What does your source code look like which uses that stream import?

Does `new URL("index.html", import.meta.url)` cause the same problem?

> Also, this PR needs to be tested before it is merged Why the **_strong_** emphasis 😄 ? Any particular cases where you're not sure whether they work? Would it...

In case you didn't notice, 27 existing tests are also failing. e.g. ``` 1) hmr hmr server should emit an HMR update for the file that changed: Error: Timeout of...

esbuild and Flow also fail, so you need these commands to see all the CI failures locally: ```sh yarn lint yarn flow yarn test:integration # run all tests yarn test:integration...

1. Don't use `"module"` as the target name unless you're literally building some JS library to publish to npm/.... Call it "app" or anything other than module/main/browser 2. You imported...

That code is actually unused, the REPL uses this function: https://github.com/parcel-bundler/parcel/blob/e14784dedd0072b82e9c137736312e5c2f8ca0fb/packages/dev/repl/src/parcel/yarn.js#L61