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

@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.

@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
@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
Copy link
Contributor Author

dyniec commented Jun 8, 2024

fixed formatting

@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