Skip to content

fix(formatter): preserve comments - #768

Merged
psteinroe merged 3 commits into
supabase-community:mainfrom
gintsgints:preserve_comments
Jul 21, 2026
Merged

fix(formatter): preserve comments#768
psteinroe merged 3 commits into
supabase-community:mainfrom
gintsgints:preserve_comments

Conversation

@gintsgints

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Bug fix #767

What is the current behavior?

Comments are not preserved on format

What is the new behavior?

Comments are reinserted into formated statements

Additional context

Code is AI generated, but reviewed by me. Tests also are created and looks good.

@psteinroe

Copy link
Copy Markdown
Collaborator

Thanks for the contribution! I am touching grass at the moment, but will come back to this next week. 🫶

@psteinroe psteinroe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks for the contribution! i would love to actually support comments at some point but this should at least unlock usage of the formatter for most people.

one thing: could you adapt the note in the formatter docs to explain the new behaviour?

thanks!

@gintsgints

Copy link
Copy Markdown
Contributor Author

Did small update on formatting.md. If you think more info is needed, let me know.

@psteinroe psteinroe changed the title Preserve comments fix(formatter): preserve comments Jul 21, 2026
@psteinroe
psteinroe merged commit a4d6b2e into supabase-community:main Jul 21, 2026
9 checks passed
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.

format removes comments

2 participants