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

Add codeactions for cabal field names #3273

Merged
merged 4 commits into from
Jul 11, 2024

Conversation

dyniec
Copy link
Contributor

@dyniec dyniec commented Oct 8, 2022

Depends on #3268 (and transitively on #2954).

Adds codeactions for misspelled field names in cabal files.
This change moves some inner workings of completions change, to use same logic (and same data).
examples:
Screenshot_20221008_153653
Screenshot_20221008_153707

@fendor fendor force-pushed the cabal_field_codeactions branch from 84a291a to b6282fb Compare May 7, 2024 15:57
@dyniec dyniec requested a review from fendor as a code owner May 7, 2024 15:57
@fendor fendor added the status: needs review This PR is ready for review label May 7, 2024
@fendor fendor force-pushed the cabal_field_codeactions branch 2 times, most recently from e2201ec to a3d902a Compare May 7, 2024 15:59
@fendor
Copy link
Collaborator

fendor commented May 7, 2024

I rebased the PR and plan to push this over the finish line.

Only tests are missing.

@fendor fendor force-pushed the cabal_field_codeactions branch from a3d902a to 2327c14 Compare May 7, 2024 16:13
@Kleidukos
Copy link
Member

Thanks a lot @dyniec and @fendor!

Copy link
Member

@Kleidukos Kleidukos left a comment

Choose a reason for hiding this comment

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

This is obviously a great addition, but it deserves a changelog entry. :)

@fendor
Copy link
Collaborator

fendor commented May 27, 2024

@Kleidukos I was wondering, where do we have changelog entries in HLS? :D

@Kleidukos
Copy link
Member

I know the cabal plugin has a changelog

@fendor
Copy link
Collaborator

fendor commented May 27, 2024

you are indeed correct!

@fendor fendor force-pushed the cabal_field_codeactions branch 2 times, most recently from f4422a3 to 817a74b Compare May 27, 2024 19:27
@fendor fendor force-pushed the cabal_field_codeactions branch from 817a74b to 228bad7 Compare May 28, 2024 08:03
@dyniec dyniec force-pushed the cabal_field_codeactions branch from 228bad7 to 85fc20e Compare June 8, 2024 14:19
@dyniec
Copy link
Contributor Author

dyniec commented Jun 8, 2024

Rebased on top of current master branch. Changelogs are removed so no need to add entry

@fendor fendor enabled auto-merge (squash) June 8, 2024 15:11
auto-merge was automatically disabled June 8, 2024 16:18

Head branch was pushed to by a user without write access

@dyniec dyniec force-pushed the cabal_field_codeactions branch from 85fc20e to 9ff4df1 Compare June 8, 2024 16:18
@dyniec
Copy link
Contributor Author

dyniec commented Jun 8, 2024

fixed formatting

@fendor fendor force-pushed the cabal_field_codeactions branch from 9ff4df1 to bf2e1fa Compare June 9, 2024 13:48
@VeryMilkyJoe VeryMilkyJoe force-pushed the cabal_field_codeactions branch from bf2e1fa to 3055435 Compare June 10, 2024 08:05
@fendor fendor force-pushed the cabal_field_codeactions branch from 3055435 to e6b6cd6 Compare June 12, 2024 20:49
@VeryMilkyJoe VeryMilkyJoe force-pushed the cabal_field_codeactions branch from e6b6cd6 to 200c8df Compare July 11, 2024 08:58
@VeryMilkyJoe VeryMilkyJoe force-pushed the cabal_field_codeactions branch from 200c8df to 26d3622 Compare July 11, 2024 09:17
@fendor fendor requested a review from Kleidukos July 11, 2024 09:42
@fendor fendor dismissed Kleidukos’s stale review July 11, 2024 09:43

We deleted changelogs and the review is about a missing changelog entries

@VeryMilkyJoe VeryMilkyJoe enabled auto-merge (squash) July 11, 2024 14:48
@VeryMilkyJoe VeryMilkyJoe merged commit ce486f7 into haskell:master Jul 11, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs review This PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants