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

[lsp] Add support for ignored files in ls formatting #845

Merged

Conversation

charlieegan3
Copy link
Member

Adding functionality to manage the contents of ignored files when formatting documents.

Addresses some issues with ignored files missed in #837

@charlieegan3 charlieegan3 force-pushed the handle-formatting-for-ignored branch from e1361e0 to f2944ec Compare June 17, 2024 13:41
@charlieegan3 charlieegan3 changed the title Add support for ignored files in ls formatting [lsp] Add support for ignored files in ls formatting Jun 17, 2024
Copy link
Member

@anderseknert anderseknert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! We can extend this to cover more "local" operations later if ever requested.

README.md Outdated Show resolved Hide resolved
@charlieegan3 charlieegan3 force-pushed the handle-formatting-for-ignored branch 2 times, most recently from de725f7 to 1c5e4cf Compare June 17, 2024 13:51
Adding functionality to manage the contents of ignored files when
formatting documents.

Signed-off-by: Charlie Egan <charlie@styra.com>
@charlieegan3 charlieegan3 force-pushed the handle-formatting-for-ignored branch from 1c5e4cf to f45f778 Compare June 17, 2024 13:58
@charlieegan3 charlieegan3 merged commit 8dd71c8 into StyraInc:main Jun 17, 2024
3 checks passed
@charlieegan3 charlieegan3 deleted the handle-formatting-for-ignored branch June 17, 2024 14:04
srenatus pushed a commit to srenatus/regal that referenced this pull request Oct 1, 2024
Adding functionality to manage the contents of ignored files when
formatting documents.

Signed-off-by: Charlie Egan <charlie@styra.com>
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