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

fix(fixer): restore the addition of the GitModified filter option #134

Open
wants to merge 1 commit into
base: develop-1.0.0
Choose a base branch
from

Conversation

valeryan
Copy link
Owner

@valeryan valeryan commented Sep 2, 2024

Not sure if it was correct to remove the filter. I need to test if its correct to use git modified here or remove and cleanup document range.

The logic for git modified was added here: #54
This logic was removed here: https://github.com/valeryan/vscode-phpsab/pull/55/files#diff-8252e32d33853b0c73ac8a3449d744d96025b7aa6926201d9ae16c4a6d3c079bL113

But did I remove it by mistake or for reason? The world may never know.

image

Not sure if it was correct to remove the filter. I need to test if its correct to use git modified
here or remove and cleanup document range.
@jonathanbossenger
Copy link
Collaborator

@valeryan I don't use VS Code regularly, what's the best way to test the functionality added back in this PR.

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