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

-t/--target option #106

Merged
merged 1 commit into from Mar 8, 2017
Merged

-t/--target option #106

merged 1 commit into from Mar 8, 2017

Conversation

@dahlia
Copy link
Member

@dahlia dahlia commented Mar 7, 2017

Although #99 made Nirum's internals possible to extend targets, it didn't expose its extensiblity to the user interface (CLI).

This patch added -t/--target option to specify the target language of the object. Also it prints the list of all available targets.

@dahlia dahlia self-assigned this Mar 7, 2017
@dahlia dahlia requested review from kanghyojun, yjroot, Kroisse, AiOO and heejongahn Mar 7, 2017
@@ -61,6 +61,7 @@ For more infomration, use `--help` option:

Common flags:
-o --output-dir=DIR The directory to place object files
-t --target=TARGET The target language. Available targets: python

This comment has been minimized.

@heejongahn

heejongahn Mar 8, 2017
Contributor

(시무룩)

@dahlia dahlia merged commit be1dbe3 into nirum-lang:master Mar 8, 2017
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants