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

Feature request: watch mode #22

Open
mugli opened this issue May 27, 2021 · 1 comment
Open

Feature request: watch mode #22

mugli opened this issue May 27, 2021 · 1 comment

Comments

@mugli
Copy link

mugli commented May 27, 2021

golds right now does not pick up changes made in the packages it has already parsed, even if we refresh the browser. Please consider adding a watch mode to reflect the changes on the filesystem. I think it would be very helpful for package authors.

@zigo101
Copy link
Collaborator

zigo101 commented May 27, 2021

Yes, Golds is mainly used to analyze a snapshot of a code base.
Golds now maintains a lot of analyzed information in memory.
A simple code change might invalidate much of the information.
Rebuilding the information needs some time so might cause some obvious lags.

To avoid the lags, some analyzing work needs to be lazy executed as needed.
The code needs to be refactored like go-lsp.
I have not a clear idea on how to make the refactoring now,
but I think it might be possible.
I will add this to the todo list now.

On the other hand, the refactoring needs much effort and time to finish.
Maybe it would be better to develop another Go app to support the watch feature.

I really don't know which route is better now.

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

No branches or pull requests

2 participants