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

luna run --target fails to accept directory with trailing separator #416

Closed
mwu-tow opened this issue Jun 26, 2019 · 0 comments
Closed

luna run --target fails to accept directory with trailing separator #416

mwu-tow opened this issue Jun 26, 2019 · 0 comments
Labels
d-hard Difficulty: significant prior knowledge required p-high Should be completed in the next sprint

Comments

@mwu-tow
Copy link
Contributor

mwu-tow commented Jun 26, 2019

General Summary

Shell luna help message says:

Usage: luna run [--target FILE/FOLDER]
  Execute a luna package, or standalone file.

Available options:
  --target FILE/FOLDER     Execute FILE/FOLDER in interpreted mode.

Steps to Reproduce

Run target with path to directory with trailing separator:
luna run --target c:\dev\Dataframes\

Expected Result

Interpreter executes project from the given location.

Actual Result

Error: InvalidAbsFile "c:\\dev\\Dataframes\\\\"

Luna Version

e927f7c
@mwu-tow mwu-tow changed the title luna run --target fails to accept luna file or directory with trailing separator luna run --target fails to accept directory with trailing separator Jun 26, 2019
@iamrecursion iamrecursion added Category: Runtime d-hard Difficulty: significant prior knowledge required p-high Should be completed in the next sprint labels Jun 27, 2019
@joenash joenash closed this as completed Jun 23, 2020
iamrecursion pushed a commit that referenced this issue Jun 23, 2020
@mwu-tow
HasSpan and HasRepr implementation for AST (#416)
2f015c7 
Provides implementation of `HasSpan` and (newly introduced) `HasRepr` for all relevant Ast types.
Implements #379.

Wherever feasible, I tried to reduce boilerplate with macros. Certainly more can be done in that regard but that'd require spending more time on this task than we allocated.
somebody1234 added a commit that referenced this issue May 3, 2023
@somebody1234
Fix cloud-v2/#416
cb22d7c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
d-hard Difficulty: significant prior knowledge required p-high Should be completed in the next sprint
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mwu-tow @joenash @iamrecursion