Skip to content

Re-add org support via orgmode plugin. - #464

Open
schilkp wants to merge 3 commits into
stevearc:masterfrom
schilkp:schilkp/orgmode
Open

Re-add org support via orgmode plugin.#464
schilkp wants to merge 3 commits into
stevearc:masterfrom
schilkp:schilkp/orgmode

Conversation

@schilkp

@schilkp schilkp commented Apr 30, 2025

Copy link
Copy Markdown

See #463 for context.

@schilkp

schilkp commented May 9, 2025

Copy link
Copy Markdown
Author

I have started adding other small fixes to this branch. Happy to pull them into separate PRs if org is something you see aerial supporting in the future.

This fix hides links in titles, which are usually concealed in the normal editor.

In other words, render:

* Level1_3 [[id:b2c99868-6014-4e81-a484-a4a7f7e62128][link1]] [[id:b2c99868-6014-4e81-a484-a4a7f7e62128][link2]]

as

* Level1_3 link1 link2

@stevearc

Copy link
Copy Markdown
Owner

Looks like CI isn't picking up on the treesitter parser for some reason. Once you get CI to pass, I'm happy to merge!

schilkp added 2 commits July 28, 2026 14:43
…ng testing

The `org` TS parser was dropped from `nvim-treesitter` due to it being
unmaintained. However, the popular `orgmode` plugin maintains a fork
that it installs. This installs the plugin in the test harness script to
allow testing of the org queries.

Also converts the expected test output to the new format.
@schilkp
schilkp marked this pull request as ready for review July 28, 2026 12:56
@schilkp

schilkp commented Jul 28, 2026

Copy link
Copy Markdown
Author

Sorry for the (year long) delay!

I was using my fork with the fix and forgot about upstreaming...

This has been updated to use the new testing snapshot format and CI passes when I run it on github-hosted runners on my fork:
https://github.com/schilkp/aerial.nvim/actions/runs/30360811653/job/90280761031

Do you mind re-running CI to check if this is good-to-go?

Cheers!

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