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

feat: Implements go to references #41

Merged
merged 9 commits into from
Nov 18, 2024
Merged

feat: Implements go to references #41

merged 9 commits into from
Nov 18, 2024

Conversation

nabekou29
Copy link
Owner

@nabekou29 nabekou29 commented Nov 13, 2024

Generated by Copilot

This pull request to the lua/js-i18n module includes significant changes to improve the handling of source inputs and enhance the functionality of the language server protocol (LSP). The most important changes are grouped into improvements to query handling, LSP enhancements, and utility adjustments.

Improvements to query handling:

  • lua/js-i18n/analyzer.lua: Replaced bufnr with a more flexible source parameter, which can be either a buffer number or a string source. This change affects multiple functions, including get_node_for_key, parse_get_t, parse_call_t, and find_call_t_expressions. [1] [2] [3] [4] [5]

LSP enhancements:

Utility adjustments:

These changes collectively enhance the flexibility and performance of the js-i18n module, particularly in handling various source inputs and improving LSP functionalities.

@nabekou29 nabekou29 self-assigned this Nov 13, 2024
@nabekou29 nabekou29 force-pushed the feat/go_to_referances branch 2 times, most recently from 943d48b to 48c20f9 Compare November 17, 2024 13:46
@nabekou29 nabekou29 force-pushed the feat/go_to_referances branch from ca5d171 to bd2fb6e Compare November 17, 2024 16:22
@nabekou29 nabekou29 marked this pull request as ready for review November 18, 2024 05:42
@nabekou29 nabekou29 merged commit 19c7d6d into main Nov 18, 2024
1 check passed
@nabekou29 nabekou29 deleted the feat/go_to_referances branch November 18, 2024 05:59
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.

Add go to reference
1 participant