Skip to content

Conversation

@doriath
Copy link
Contributor

@doriath doriath commented Dec 16, 2024

It is allowed to pass a path as bareword in nu, e.g. in ls or use commands.

@kubouch
Copy link
Contributor

kubouch commented Dec 16, 2024

The new lexer will replace this code, so let's not land it. I'll keep it open, though, as a reminder to make sure paths work with the new lexer.

The new lexer will significantly change / replace both the lexing and some of the parsing parts, so if you're looking for contributions, it's best to stick with the Resolver / Typechecker for now.

@doriath
Copy link
Contributor Author

doriath commented Dec 16, 2024

I wanted to look into modules and overlays, and the first thing I realized is that I can't do use foo.nu * because foo.nu does not parse correct.

  • If the main concern is that this completely goes away with new parser, I would still just merge it
  • if this will cause issues for you with merging the new parser/lexer, then indeed let's not merge (and I will continue iterating on modules / overlays on top of it, without submitting it to main branch)

It is allowed to pass a path as bareword in nu, e.g. in `ls` or `use`
commands.
@kubouch
Copy link
Contributor

kubouch commented Dec 28, 2024

Resolved by #31

@kubouch kubouch closed this Dec 28, 2024
@doriath doriath deleted the paths branch January 2, 2025 06:49
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