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

Add Nix support #3954

Merged
merged 1 commit into from
Nov 26, 2024
Merged

Add Nix support #3954

merged 1 commit into from
Nov 26, 2024

Conversation

techee
Copy link
Member

@techee techee commented Sep 16, 2024

@frlan Christmas ;-)

This is a work-in-progress Nix lexer I plan to submit to Lexilla but I'd like some feedback from @frlan as he's the most proficient Nix user around. Things should mostly work, I'm just aware of a problem regarding folding when a block ends and is followed by a multiline string on the same line:

{
    key = value;
} ''
    something
''

This is something I have to ask Neil how to handle the best way.

Otherwise I'm not aware of any other issues and I'm interested in any problems that appear. Especially if strings, escaping, and string interpolations work as expected and in general if the highlighting seems correct or something is missing.

Also, for now, the config file doesn't contain correct build commands - @frlan eventually this is something I'll need from you for Geany.

@frlan
Copy link
Member

frlan commented Sep 27, 2024

I'm checking in more daily use now

@techee
Copy link
Member Author

techee commented Oct 12, 2024

Talked with Frank in person today and it seems to work for him so submitting upstream.

@techee techee changed the title WIP: Nix lexer Add Nix support Nov 1, 2024
@techee
Copy link
Member Author

techee commented Nov 1, 2024

I've updated the lexer to the version merged into Lexilla. This PR is completed from my perspective.

@techee
Copy link
Member Author

techee commented Nov 20, 2024

Alright, I rebased this PR on top of master, re-checked the patch, re-tested and everything seems to be all right.

So unless there are any objections, I'll merge it in about a week.

@techee techee merged commit 22a8a1f into geany:master Nov 26, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants