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

Fix block attribute doc comment reformatting #3030

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

tomhoule
Copy link
Contributor

@tomhoule tomhoule commented Jul 4, 2022

Fix block attribute doc comment reformatting

This is achieved by a massive cleanup and simplification of the
AST reformatter.

  • The comments attached to block attributes will follow them after
    reformatting.
  • Enum attributes do not get reformatted with an extra leading space
    anymore
  • Empty lines at beginning and end of blocks are reformatted away.

closes prisma/language-tools#1186

Also included:

  • diagnostics: delete dead warnings code and stop using thiserror —
    compile times will thank us.

@tomhoule tomhoule force-pushed the schema-ast/pure-reformatter branch 2 times, most recently from bffb615 to 8cbc3fd Compare July 4, 2022 16:05
@tomhoule tomhoule changed the title (wip) fix comment reformatting Fix block attribute doc comment reformatting Jul 4, 2022
@tomhoule tomhoule force-pushed the schema-ast/pure-reformatter branch 2 times, most recently from 7b0b77c to b1d94f6 Compare July 4, 2022 17:18
@tomhoule tomhoule added this to the 4.1.0 milestone Jul 4, 2022
@tomhoule tomhoule force-pushed the schema-ast/pure-reformatter branch 4 times, most recently from 63ab71a to a1b2a66 Compare July 5, 2022 04:26
This is achieved by a massive cleanup and simplification of the
AST reformatter.

- The comments attached to block attributes will follow them after
  reformatting.
- Enum attributes do not get reformatted with an extra leading space
  anymore
- Empty lines at beginning and end of blocks are reformatted away.

closes prisma/language-tools#1186

Also included:

- diagnostics: delete dead warnings code and stop using thiserror —
  compile times will thank us.
@tomhoule tomhoule force-pushed the schema-ast/pure-reformatter branch from a1b2a66 to c648aee Compare July 5, 2022 04:34
@tomhoule tomhoule marked this pull request as ready for review July 5, 2022 04:45
@tomhoule tomhoule requested a review from a team as a code owner July 5, 2022 04:45
@tomhoule
Copy link
Contributor Author

tomhoule commented Jul 5, 2022

Reviewed in person with @pimeys .

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.

Cannot add comments to my @@unique constraints
1 participant