Skip to content

Conversation

@SVilgelm
Copy link
Member

Conver the SuggesterFixes to Replacement by:

  1. applying the fixes to a file in memory
  2. creating a diff by comparing with original file
  3. using ExtractIssuesFromPatch function to create the issues

@SVilgelm SVilgelm requested a review from ldez February 22, 2022 04:43
@ldez ldez self-assigned this Feb 22, 2022
@ldez ldez added the enhancement New feature or improvement label Feb 22, 2022
@SVilgelm SVilgelm force-pushed the sv/support-suggested-fixes branch 2 times, most recently from 89b7513 to 9ed5793 Compare February 22, 2022 16:01
@SVilgelm
Copy link
Member Author

@ldez Ok, the PR is ready. I understand that the solution is not the best possible, but I still don't understand how to manually create a Replacement object for a TextEdit. This is kind of workaround

PS: gci does not provide SuggestedFixes :(

@SVilgelm SVilgelm force-pushed the sv/support-suggested-fixes branch from 9ed5793 to 1371f5c Compare February 23, 2022 17:27
Conver the SuggesterFixes to Replacement by:
1. applying the fixes to a file in memory
2. creating a diff by comparing with original file
3. using `ExtractIssuesFromPatch` function to create the issues
@SVilgelm SVilgelm force-pushed the sv/support-suggested-fixes branch from 1371f5c to 43d956f Compare February 24, 2022 20:47
@SVilgelm SVilgelm force-pushed the sv/support-suggested-fixes branch from 43d956f to d766d2c Compare February 24, 2022 21:00
@ldez ldez added the blocked Need's direct action from maintainer label Mar 27, 2022
@nightlyone

This comment was marked as off-topic.

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

Labels

area: auto-fix blocked Need's direct action from maintainer enhancement New feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants