-
Notifications
You must be signed in to change notification settings - Fork 522
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
Rename/move ts_devserver #1082
Comments
You caught our organizational concern! if I were to name this today, I'd call it concatjs_devserver since it's tied to that particular bundler (and requires named AMD/UMD inputs or goog.module which is unusual) We are thinking of some renames internally too. Googlers: http://b/140889721 |
Let's rename to |
…l/concatjs BREAKING CHANGE: users need to change their load statements for ts_devserver Fixes bazel-contrib#1082
…l/concatjs BREAKING CHANGE: users need to change their load statements for ts_devserver Fixes bazel-contrib#1082
…l/concatjs BREAKING CHANGE: users need to change their load statements for ts_devserver Fixes bazel-contrib#1082
…ncatjs BREAKING CHANGE: users need to change their load statements for ts_devserver Fixes bazel-contrib#1082
…ncatjs BREAKING CHANGE: users need to change their load statements for ts_devserver Fixes #1082
landed in #2239 |
🚀 feature request
Nit, but "ts_devserver" is a rather misleading name, as it doesn't touch TypeScript at all. Ideally, it'd be named something like
js_devserver
.Also, funnily enough, it's in rules_typescript. (I suspect this simply the world being exposed to some internal Google organizational concern?)
The text was updated successfully, but these errors were encountered: