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

Move everything from cairo_lang_semantic::lsp_helpers to LsSemanticGroup #6137

Open
mkaput opened this issue Aug 5, 2024 · 0 comments
Open

Move everything from cairo_lang_semantic::lsp_helpers to LsSemanticGroup #6137

mkaput opened this issue Aug 5, 2024 · 0 comments
Labels
ide This issue refers to CairoLS or editor extensions

Comments

@mkaput
Copy link
Contributor

mkaput commented Aug 5, 2024 •
edited
Loading

As title says, LsSemanticGroup is the better place for these functions
@mkaput mkaput added the ide This issue refers to CairoLS or editor extensions label Aug 5, 2024
@piotmag769 piotmag769 linked a pull request Aug 6, 2024 that will close this issue
LS: fix panic in lsp_helper and move its logic to LsSemanticGroup #6159
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ide This issue refers to CairoLS or editor extensions
Projects
CairoLS
Status: Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

LS: fix panic in lsp_helper and move its logic to LsSemanticGroup starkware-libs/cairo
2 participants
@mkaput @piotmag769