Skip to content

Conversation

@janopae
Copy link
Member

@janopae janopae commented Sep 9, 2025

Adds support for TypeScript files to the Webpack bundling task.

The gulp task including the modifications from this PR is already in use in https://github.com/webfactory/webfactory-de-projekt-controlling, which can be set back to using the upstream task after this has been merged.

Apart from being necessary for our Projekt Controlling app, I'd like to make it easier to use TypeScript in webfactory projects to encourage its use.

It might be "cleaner" to open the gulp preset for downstream Webpack config modifications rather than adding TypeScript upstream. However, this is already how we decided to handle Svelte support, which is used by a similarly small amount of webfactory projects, and it seems to be easier right now – the downsides of requiring TypeScript upstream should not be this big, given this is just a dev dependency.

That doesn't mean I wouldn't be positive designing an interface for additional webpack configuration from downstream, and maybe bringing the syntax expected in gulp-config.js closer to the well documented Webpack config format.

@janopae janopae changed the title Support TypeScript files as bundle sources Support TypeScript files as bundle sources (Case 199344) Sep 9, 2025
@janopae janopae requested a review from polarbirke September 9, 2025 11:49
Copy link
Member

@polarbirke polarbirke left a comment

Choose a reason for hiding this comment

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

Let's keep following the "it just (magically) works" approach for the time being 👍

The next big step will probably involve moving away from Webpack and/or Gulp in favor of… something1.

Footnotes

  1. Probable contenders are Vite and esbuild instead of Webpack; the future of Gulp is not as clear-cut, but perhaps simple npm scripts could be a way forward. The problem is not Gulp itself, but the fact that its plugin ecosystem seems to be on a steady decline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants