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

LS: notify to restart server on config change #38

Open
mkaput opened this issue Dec 18, 2024 · 1 comment
Open

LS: notify to restart server on config change #38

mkaput opened this issue Dec 18, 2024 · 1 comment

Comments

@mkaput
Copy link
Member

mkaput commented Dec 18, 2024

@Draggu commented on 13.12.2024, 21:50:32:

Copy this RA behaviour image
@mkaput
Copy link
Member Author

mkaput commented Dec 18, 2024

@mkaput commented on 16.12.2024, 06:29:17:

Mind that we can be smarter than RA and avoid restarts for some configuration options (for example trace diagnostics). RA takes the entire config as a custom input JSON object and has no means for being notified on changes, while we use workspace/configuration method from LSP.

@mkaput mkaput moved this from Triage to Backlog in Cairo Language Services Dec 18, 2024
@mkaput mkaput marked this as a duplicate of #43 Mar 13, 2025
@mkaput mkaput moved this from Backlog LS to Idea aka Deep Backlog in Cairo Language Services Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Cairo Language Services
Status: Sranie
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mkaput