Merge ~mpontillo/maas:remove-es6-for-now into maas:master
Proposed by
Mike Pontillo
Status: | Merged |
---|---|
Approved by: | Mike Pontillo |
Approved revision: | 7620d84472f610759334f40660346d3bd62f73e7 |
Merge reported by: | MAAS Lander |
Merged at revision: | not available |
Proposed branch: | ~mpontillo/maas:remove-es6-for-now |
Merge into: | maas:master |
Diff against target: |
24 lines (+3/-3) 1 file modified
src/maasserver/static/js/angular/directives/maas_obj_form.js (+3/-3) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Lee Trager (community) | Approve | ||
Review via email: mp+342485@code.launchpad.net |
Commit message
Remove ES6 (for now).
Description of the change
The team decided that to stick to ES5 JavaScript for now, to maintain consistency with existing code.
While I would personally prefer to keep the existing ES6 and continue to phase it in, this branch removes the ES6 that I have so far intentionally added.
To post a comment you must log in.
We should discuss allowing ES6 for next cycle as I agree phasing ES6 in would be a good way forward.