Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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.

# for free to join this conversation on GitHub. Already have an account? # 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