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

Added support for module.exports #76

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

richardvanthof
Copy link

Now the program just returns the transpiled file if no output value has been given.
This is useful when integrating this tool into gulp pipelines (in combination with gulp-exec).

@ff6347
Copy link
Member

ff6347 commented Jul 26, 2023

Hi @richardvanthof Thank you for your contribution.
This tool is a blast from the past. Nice to see that it is still in use.

I tried your branch but when I run:

node ./index.js --script test/src/indesign.js
Running extendscriptr with following options:
--script: test/src/indesign.js
--output: undefined
--target: undefined (optional)
--targetengine: undefined (optional)

The only output I get is the default option listing and no output to stdout from the compiled script. I'm I doing something wrong?

Also some things I would love to see before merging.

  • Some kind of test
  • Update of the options
  • Update of the README

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