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

Improving auto-completion by using the "include" subtype #229

Open
jdose100 opened this issue Feb 14, 2025 · 3 comments
Open

Improving auto-completion by using the "include" subtype #229

jdose100 opened this issue Feb 14, 2025 · 3 comments
Labels
0.10.1 To be completed before the 0.10.1 release enhancement New feature or request

Comments

@jdose100
Copy link

I recently started working with this lsp and came across the fact that it does not fully support the include directive. That is, when you include another file, auto-completion does not offer new characters defined in another file. I would like to see this feature here, as it is very convenient.

@WillLillis
Copy link
Collaborator

WillLillis commented Feb 14, 2025

This should be doable, but will have to be implemented on an assembler-by-assembler basis. Can I ask which one you're using?

I'll make a pass on implementing this after our new testing framework (see #228) is in place, as we currently have no means to test with multiple files.

@WillLillis WillLillis added enhancement New feature or request 0.10.1 To be completed before the 0.10.1 release labels Feb 14, 2025
@jdose100
Copy link
Author

Hello, I'm currently using GNU assembler. But I also expect fasm support

@WillLillis
Copy link
Collaborator

Hello, I'm currently using GNU assembler. But I also expect fasm support

I'll make sure to cover FASM too once it's added (#218).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.10.1 To be completed before the 0.10.1 release enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants