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

typescript: Undocumented outputToFilesystem option emits warning if not present #1227

Closed
2 of 4 tasks
danopia opened this issue Jul 24, 2022 · 12 comments
Closed
2 of 4 tasks

Comments

@danopia
Copy link

danopia commented Jul 24, 2022

  • Rollup Plugin Name: @rollup/plugin-typescript
  • Rollup Plugin Version: 8.3.3

Documentation Is:

  • Missing
  • Needed
  • Confusing
  • Not Sure?

Please Explain in Detail...

When configured minimally (so, using defaults), the plugin is printing a warning:

(!) Plugin typescript: @rollup/plugin-typescript: outputToFilesystem option is defaulting to true.

This option is not documented on the README and the message doesn't have any further information. It's not clear what the options are to remove the warning.

Your Proposal for Changes

  1. The message could be more detailed: What did I do to introduce this warning? Is it related to a tsconfig.json option? Is there documentation it can link to, or a recommended action to take?
  2. The option should be added to the other options in the plugin documentation. When is it needed? What are the reasons to use either false or true here? Are there other possible values? Since what version has this flag been soft-required?

References

@danopia danopia changed the title Undocumented outputToFilesystem option emits warning if not present typescript: Undocumented outputToFilesystem option emits warning if not present Jul 24, 2022
@shellscape
Copy link
Collaborator

Happy to review a PR with the changes you'd like to see.

@mistic100
Copy link

@shellscape the question is more : what needs to be done to remove the warning ?

@vividlearningsystems
Copy link

I just ran into this today when trying to turn on incremental builds. When "incremental" is set to true in tsconfig this warning appears. At the same time a new ".rollup.cache" folder is created in project root. Could this warning be related to that?

@stale stale bot added the x⁷ ⋅ stale label Dec 16, 2022
@stale
Copy link

stale bot commented Dec 20, 2022

Hey folks. This issue hasn't received any traction for 60 days, so we're going to close this for housekeeping. If this is still an ongoing issue, please do consider contributing a Pull Request to resolve it. Further discussion is always welcome even with the issue closed. If anything actionable is posted in the comments, we'll consider reopening it.

@stale stale bot closed this as completed Dec 20, 2022
@ctretyak
Copy link

I think it should be reopened

@Bloodsucker
Copy link

This ticket should be reopen and I believe it is related to Typescript Project References.

@ericmasiello
Copy link

➕ Can this be reopened? This is still a problem

@shellscape
Copy link
Collaborator

people, please read the bot message. it will be reopened when the community steps up to help.

@vpalos
Copy link

vpalos commented Mar 20, 2023

As a workaround, if you don't really need incremental compiling for TypeScript, just set incremental to false in your tsconfig.json. This did the trick for me. Thanks @vividlearningsystems for pointing that out.

@quisido
Copy link

quisido commented Sep 3, 2023

For reference, here is the IntelliSense for the property:

When set to false, force non-cached files to always be emitted in the output directory.output If not set, will default to true with a warning.

@segevfiner
Copy link

Generally the behavior without this option set is technically correct despite the warning. The file outputted outside the bundle, the tsbuildinfo is not really part of the output, and as such, writing it using Rollup's emitFile doesn't matter and it can be written to the disk directly. But, this presents a warning to the user. On the other hand, enabling outputToFilesystem will write all output directly to the filesystem, bypassing Rollup, which is not desired. So maybe we just need an option to disable the warning, or otherwise not output it for files that are safe to write directly to the filesystem?

@NotYoojun
Copy link

Please reopen this issue. I really need incremental to be true for my project since it takes a really long time when building.

jthrilly added a commit to complexdatacollective/network-canvas-monorepo that referenced this issue Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests