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

Remove decorations when removing a file from a project #1788

Merged

Conversation

MangelMaxime
Copy link
Contributor

Preview:

remove_decoration_when_removing_file.mp4

This PR remove the decorations added by the PipelineHints and LineLens when removing a file.

With this PR and when ionide/FsAutoComplete#1005 will be merged we should have everything cleaned when removing a file.

ionide/FsAutoComplete#1005 is needed to:

  • Clean the diagnostics
  • Not have the file still considered part of the project when re-opening it
    With the FSAC PR, if you re-open the file then it still consider that it can access others files content which is wrong :)

@baronfel
Copy link
Contributor

Thanks for this @MangelMaxime - I've released an FSAC with your removal fix and will include this whole package in tonight's Ionide release.

@baronfel baronfel merged commit 7623c91 into ionide:main Oct 20, 2022
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.

2 participants