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 VHDL extension #729

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add VHDL extension #729

wants to merge 1 commit into from

Conversation

rapgenic
Copy link

@rapgenic rapgenic commented May 15, 2024

This PR implements an extension for the VHDL language using

This is my first attempt at creating an extension, so any feedback is appreciated!

I'm also marking this as a draft, because I cannot test MacOS and Windows, plus I'd like to use it for a few days to iron out any problems (I wrote it in a few hours, so there might be some rough mistakes...).

Copy link

cla-bot bot commented May 15, 2024

We require contributors to sign our Contributor License Agreement, and we don't have @rapgenic on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@rapgenic
Copy link
Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label May 15, 2024
Copy link

cla-bot bot commented May 15, 2024

The cla-bot has been summoned, and re-checked this pull request!

@rapgenic rapgenic marked this pull request as draft May 15, 2024 18:20
@rapgenic
Copy link
Author

I am having some problems with symbol outline and multiple variable definitions in a single statement, as per the following example:

variable a, b : Integer;

When showing the outline, the b variable appears as a child of the a variable, see the following picture:

image

The current tree-sitter query for that node is:

(variable_declaration
    "variable" @context
    (identifier_list (identifier) @name)
) @item

I cannot find a way to make them appear as two distinct variables, with the correct alignment...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant