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.

Sign up for GitHub

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

Jump to bottom

Add CLI Interface to Pest to Ion Converter #36

Closed
almann opened this issue Jun 7, 2021 · 0 comments · Fixed by #44
Closed

Add CLI Interface to Pest to Ion Converter #36

almann opened this issue Jun 7, 2021 · 0 comments · Fixed by #44
Assignees
@almann
Labels
enhancement New feature or request

Comments

@almann
Copy link
Contributor

almann commented Jun 7, 2021

As #35 implies, we will have a library to convert Pest grammars to Ion. We should provide a CLI interface for that.
@almann almann added the enhancement New feature or request label Jun 7, 2021
@almann almann self-assigned this Jun 7, 2021
almann added a commit to almann/partiql-lang-rust that referenced this issue Jun 10, 2021
@almann
Adds pest2ion command
e796995 
Adds a basic CLI driver over the `pest-ion` crate.  This driver supports
setting the input file (defaulting to STDIN), an output file (defaulting
to STDOUT), and flags around the Ion data format.

An explicit TODO is to figure out a good way to test this probably using
integration tests and spawning the command. (See partiql#42)

Resolves partiql#36.
@almann almann mentioned this issue Jun 10, 2021
Merged
almann added a commit to almann/partiql-lang-rust that referenced this issue Jun 11, 2021
@almann
Adds pest2ion command
4c026d7 
Adds a basic CLI driver over the `pest-ion` crate.  This driver supports
setting the input file (defaulting to STDIN), an output file (defaulting
to STDOUT), and flags around the Ion data format.

An explicit TODO is to figure out a good way to test this probably using
integration tests and spawning the command. (See partiql#42)

Resolves partiql#36.
almann added a commit that referenced this issue Jun 11, 2021
@almann
Adds pest2ion command (#44)
5da7928 
Adds a basic CLI driver over the `pest-ion` crate.  This driver supports
setting the input file (defaulting to STDIN), an output file (defaulting
to STDOUT), and flags around the Ion data format.

An explicit TODO is to figure out a good way to test this probably using
integration tests and spawning the command. (See #42)

Resolves #36.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@almann almann

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds pest2ion command almann/partiql-lang-rust
1 participant
@almann