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

[WIP] Introduce new code action: suggest missing fields #4325

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akshaymankar
Copy link
Contributor

Detects -Wmissing-fields warnings and suggests a code action to add all the missing fields.

Resolves #462

Pending tasks:

  • The basic test is failing because there are extra new lines
  • Add cases for HsProc
  • De-duplicate some copy-pasted code
  • Extract the code action into its own module
  • Add a few more test cases

Detects `-Wmissing-fields` warnings and suggests a code action to add all the
missing fields.

Pending tasks:
- [ ] The basic test is failing because there are extra new lines
- [ ] Add cases for `HsProc`
- [ ] De-duplicate some copy-pasted code
- [ ] Extract the code action into its own module
- [ ] Add a few more test cases
@akshaymankar
Copy link
Contributor Author

The basic test is failing because there are extra new lines

I need help with this one. I tried adding traceAst everywhere, but it still doesn't make sense to me.

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.

Feature Request: Automatic Expansion of uninitialized record fields
1 participant