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

Fix source and source-env formatting #3

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Conversation

132ikl
Copy link
Contributor

@132ikl 132ikl commented Dec 4, 2024

Fixes formatting of source and source-env.

Source:

source foo
source bar
source-env fooenv
source-env barnev

Before this PR:

source foo source bar source-env fooenv source-env barnev

After this PR:

source foo
source bar
source-env fooenv
source-env barnev

@blindFS
Copy link
Owner

blindFS commented Dec 5, 2024

Acknowledged, thanks!

One thing to note here:
the latest grammar of tree-sitter-nu has better support for newline characters, we probably won't need all those append_empty_softlines for decl/stmt in the future.

Still working on topiary-nushell to support the latest parser (some whitespace issue in the way). You can check the dev branch if you have interest.

Besides, related test case modification is much appreciated. We can leave it for now, since a major shift is coming.

@blindFS blindFS merged commit b6d8484 into blindFS:main Dec 5, 2024
1 check passed
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