Skip to content

compileFromDir (aka batch compile) #101

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

Closed
wants to merge 3 commits into from

Conversation

tomholford
Copy link

@tomholford tomholford commented Jun 22, 2017

Consider this a WIP implementation to address #16

Call compileFromDir with a list of filenames and the optional options hash (same method signature as compileFromFile)

todo:

  • Tests
  • CLI support

Copy link
Owner

@bcherny bcherny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tomholford! Is the purpose of this PR to get early feedback, or just to track your progress?

@tomholford
Copy link
Author

@bcherny Both :) Sketched out an idea to address #16 and wanted to see if it's congruent with your roadmap before investing more time into it. Further, wanted to solicit feedback on the interface - i.e. should this be available via CLI? And if so, what would the command look like?

For example:

json2ts -d schemas/models/ -R

where -d indicates the directory for compilation and -R indicates to traverse the directory recursively

@bcherny
Copy link
Owner

bcherny commented Jun 23, 2017

@tomholford See this comment and the one after it - does that seem reasonable to you?

#16 (comment)

@bcherny
Copy link
Owner

bcherny commented Jul 13, 2017

@tomholford Closing for inactivity. Feel free to reopen when/if you get back to this.

@bcherny bcherny closed this Jul 13, 2017
@tomholford
Copy link
Author

@bcherny Thanks, will re-open later

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