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

Moved the EJS templates to strings #1036

Closed

Conversation

LorneCurrie
Copy link
Contributor

Moved the EJS templates to strings so they coudl be imported as modules
Reason, is having a issue with this in AWS Lambda, where all
dependencies are compiled into one file.
Fix lint errors in:

  • lib/wsdl.js
  • test/request-response-samples/Dummy__should_handle_inline_types/wsdl_options.js
    New line added to end of Package.json

Moved the EJS templates to strings so they coudl be imported as modules
Reason, is having a issue with this in AWS Lambda, where all
dependencies are compiled into one file.
Fix lint errors in:
 - lib/wsdl.js
 - test/request-response-samples/Dummy__should_handle_inline_types/wsdl_options.js
New line added to end of Package.json
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 93.305% when pulling 7eb9ec4 on LorneCurrie:remove-ejs-file-dependancy into 9541e14 on vpulim:master.

@murbanowicz
Copy link

Please merge this. I cannot bundle my service due to this issue.

@andygreenegrass
Copy link
Contributor

ejs templates are no longer in the master branch. This was resolved in #1046

@jsdevel
Copy link
Collaborator

jsdevel commented Mar 18, 2019

thanks @andygreenegrass ! closing this issue. and thank you @LorneCurrie for originally addressing this.

@jsdevel jsdevel closed this Mar 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants