Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

lfc gives no clue how to run TypeScript code #1986

Closed
edwardalee opened this issue Sep 1, 2023 · 2 comments · Fixed by #2090
Closed

lfc gives no clue how to run TypeScript code #1986

edwardalee opened this issue Sep 1, 2023 · 2 comments · Fixed by #2090
Assignees
Labels
enhancement Enhancement of existing feature good first issue Good for newcomers typescript Related to TypeScript target
Milestone

Comments

@edwardalee
Copy link
Collaborator

When compiling a TypeScript target program, lfc used to print something like this, as shown here:

#####################################
To run the generated program, use:

    node ...path-to-project.../src-gen/Minimal/dist/Minimal.js

#####################################

It no longer prints any such thing, leaving the programmer to guess what to do next.

Better yet, it should generate a shell script in bin, as the Python target does currently, to invoke the program.

@edwardalee edwardalee added enhancement Enhancement of existing feature typescript Related to TypeScript target labels Sep 1, 2023
@axmmisaka axmmisaka self-assigned this Sep 6, 2023
@lhstrh lhstrh added the good first issue Good for newcomers label Sep 7, 2023
@lhstrh lhstrh added this to the 0.6.0 milestone Sep 27, 2023
@lhstrh lhstrh assigned hokeun and unassigned axmmisaka and lhstrh Nov 5, 2023
@hokeun
Copy link
Member

hokeun commented Nov 8, 2023

I'm looking into this.

@lhstrh
Copy link
Member

lhstrh commented Nov 8, 2023

Great! Let's opt for generating a script in bin, like we do with Python. Should be an easy fix.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement Enhancement of existing feature good first issue Good for newcomers typescript Related to TypeScript target
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants