Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#40 Loader with Nashorn #62

Merged
merged 6 commits into from Apr 8, 2016
Merged

#40 Loader with Nashorn #62

merged 6 commits into from Apr 8, 2016

Conversation

tomdye
Copy link
Member

@tomdye tomdye commented Mar 21, 2016

Addresses issue: #40
This works when nashorn is used the following way:

jjs -scripting path/to/loader.js -- main.js

Where main.js is an AMD module that may require other modules

Initial PR for discussion.

@codecov-io
Copy link

Current coverage is 93.03%

Merging #62 into master will decrease coverage by -1.32% as of deaa5be

@@            master     #62   diff @@
======================================
  Files            1       1       
  Stmts          478     488    +10
  Branches       113     117     +4
  Methods         71      72     +1
======================================
+ Hit            451     454     +3
- Partial          3       4     +1
- Missed          24      30     +6

Review entire Coverage Diff as of deaa5be

Powered by Codecov. Updated on successful CI builds.

@kitsonk kitsonk added this to the 2.0.0 milestone Mar 30, 2016
@kitsonk kitsonk self-assigned this Apr 8, 2016
@kitsonk kitsonk modified the milestones: 2016.05, 2.0.0 Apr 8, 2016
@kitsonk kitsonk merged commit d10e29f into dojo:master Apr 8, 2016
@kitsonk kitsonk modified the milestones: 2016.04, 2016.05 Apr 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants