Skip to content

Commit

Permalink
Do not expose define references in require polyfill
Browse files Browse the repository at this point in the history
Summary:
See facebook#406

Made sure the jest tests pass but didn't know a good unit test to add for this.
Closes facebook#427
Github Author: Jacob Gable <jacob.gable@gmail.com>

Test Plan:
* ./runJestTests
* start app and click around
  • Loading branch information
jgable committed Apr 1, 2015
1 parent 87599bf commit 63c2f80
Showing 1 changed file with 0 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -586,15 +586,11 @@
_register('module', 0);
_register('exports', 0);

_register('define', define);
_register('global', global);
_register('require', require);
_register('requireDynamic', require);
_register('requireLazy', requireLazy);

define.amd = {};

global.define = define;
global.require = require;
global.requireDynamic = require;
global.requireLazy = requireLazy;
Expand Down

0 comments on commit 63c2f80

Please sign in to comment.