Closed Bug 1572209 Opened 5 years ago Closed 5 years ago

Update DevTools builds to use Babel 7.x

Categories

(DevTools :: General, task)

task
Not set
normal

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: loganfsmyth, Assigned: loganfsmyth)

References

(Blocks 2 open bugs)

Details

Attachments

(4 files)

Our devtools/client/shared/build/build.js build script runs Babel 6.x, which at this point is getting pretty old.

On the Debugger we'd like to get this done because we use Flowtype, and there is some newer Flowtype syntax that is only supported on Babel 7 since it didn't exist in Babel 6.x.

Specifically, passing explicit template arguments to function calls is not supported (https://flow.org/en/docs/types/generics/#toc-supplying-type-arguments-to-callables). We are running into this because newer versions of the react-redux type definitions expect users to do:

export default connect<Props, OwnProps>(
  // ...

so that the props and are explicitly declared.

Note: Ideally whoever does this should also update the Debugger Webpack bundle scripts to also run Babel 7.x.

Blocks: 1572214
Blocks: 1574194
Assignee: nobody → lsmyth
Status: NEW → ASSIGNED
Blocks: 1575105
Pushed by lsmyth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aaa8310bb77f
Part 1: Rebuild worker bundles to pick up some missing logic. r=jlast
https://hg.mozilla.org/integration/autoland/rev/417761314937
Part 2: Upgrade debugger source-file processing to use Babel 7. r=jlast
https://hg.mozilla.org/integration/autoland/rev/63c179f03c3e
Part 3: Upgrade MC Devtools build logic to use Babel 7.x for compilation. r=jlast
Blocks: node-dx
Pushed by lsmyth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/17057707d774
Part 4: Roll back one set of accidentally-included changes. r=jlast
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: