Skip to content

Conversation

@jdufresne
Copy link
Contributor

JavaScript modules have been supported by the spec since ES6 (2015) and
the feature is available in all modern, supported browsers.

Using JavaScript modules offers advantages:

  • All scripts are loaded in strict mode, reducing unexpected edge cases
    from "sloppy mode".

  • Scripts run in isolation and no longer pollute the global namespace.
    This allows removing the IIFE from all JavaScript.

  • Browsers can optimize loading of modules.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Modules

@codecov
Copy link

codecov bot commented May 18, 2020

Codecov Report

Merging #1283 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1283   +/-   ##
=======================================
  Coverage   86.61%   86.61%           
=======================================
  Files          25       25           
  Lines        1449     1449           
  Branches      205      205           
=======================================
  Hits         1255     1255           
  Misses        142      142           
  Partials       52       52           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c7c44d9...54fc973. Read the comment docs.

@matthiask matthiask self-requested a review May 19, 2020 16:31
JavaScript modules have been supported by the spec since ES6 (2015) and
the feature is available in all modern, supported browsers.

Using JavaScript modules offers advantages:

- All scripts are loaded in strict mode, reducing unexpected edge cases
  from "sloppy mode".

- Scripts run in isolation and no longer pollute the global namespace.
  This allows removing the IIFE from all JavaScript.

- Browsers can optimize loading of modules.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Modules
@matthiask matthiask merged commit aa4d185 into django-commons:master May 20, 2020
@jdufresne jdufresne deleted the modules branch May 21, 2020 00:07
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