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

feat: Add support for the Ink programming language #12773

Merged
merged 2 commits into from
Feb 12, 2025
Merged

feat: Add support for the Ink programming language #12773

merged 2 commits into from
Feb 12, 2025

Conversation

rhizoome
Copy link
Contributor

@rhizoome rhizoome commented Feb 4, 2025 •
edited
Loading

Add support for ink by inkle. I have currently implemented basic syntax highlighting.

  • Since ink is a language for interactive fiction we use soft-wrap. The inky IDE also uses soft-wrap. This is what users expect.
  • Since ink is more a markup language and language constructs (knot, stitch, weave, gather, glue) are very different to traditional programming languages, I look the highlighting-types in highlights.scm that give a good visual impression
  • This uses my tree-sitter-ink parser, to my knowledge there is no other tree-sitter parser for ink
  • I made this because inky does not properly support unicode and I took special care to support it well

Ink Demo

@rhizoome rhizoome force-pushed the ink-syntax-highlighting branch from 1333d22 to b80d1e9 Compare February 4, 2025 22:24
nik-rev
nik-rev reviewed Feb 4, 2025
View reviewed changes
runtime/queries/ink/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/ink/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/ink/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/ink/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/ink/highlights.scm Outdated Show resolved Hide resolved
@rhizoome rhizoome force-pushed the ink-syntax-highlighting branch 3 times, most recently from 0d4ba7f to 1a1de98 Compare February 4, 2025 23:47
@the-mikedavis the-mikedavis merged commit 258e3e1 into helix-editor:master Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nik-rev nik-rev nik-rev left review comments

@the-mikedavis the-mikedavis the-mikedavis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rhizoome @the-mikedavis @nik-rev