Switch to the babel transpiler.

Review Request #7561 — Created July 31, 2015 and submitted

david
student-sonar
master
43dc444...
student-sonar

The traceur transpiler is nice and full-featured, but it has some major
problems with recent chrome updates. This change switches us over to the babel
transpiler, which means we're using the same es6 implementation on both the
backend and frontend.

Babel is unfortunately missing a couple Array features (Array.from and
Array.find), and the handling of this in fat-arrow functions is slightly
different. Babel's handling of this is actually more correct than traceur.
Babel also has more strict rules regarding super() calls in class
constructors. The bulk of this change is cleaning up those issues.

Flipped around between the calendar and user detail views. Everything seemed to
work correctly. Deployed and saw that everything worked correctly in production
too.

reviewbot
  1. Tool: PEP8 Style Checker
    Ignored Files:
        package.json
        lib/frontend/collection-view.js
        lib/frontend/config.js
        lib/frontend/group-by-collection.js
        lib/frontend/user-detail-view.js
        gulpfile.js
        lib/frontend/calendar-view.js
        lib/frontend/confirm.js
        lib/frontend/models.js
    
    
    
    Tool: Pyflakes
    Ignored Files:
        package.json
        lib/frontend/collection-view.js
        lib/frontend/config.js
        lib/frontend/group-by-collection.js
        lib/frontend/user-detail-view.js
        gulpfile.js
        lib/frontend/calendar-view.js
        lib/frontend/confirm.js
        lib/frontend/models.js
    
    
  2. 
      
brennie
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (698e596)
Loading...