Skip to content

Get pg_lake's custom types in the typedefs list for pgindent#319

Open
sfc-gh-npuka wants to merge 3 commits into
mainfrom
naisila/type_formatting
Open

Get pg_lake's custom types in the typedefs list for pgindent#319
sfc-gh-npuka wants to merge 3 commits into
mainfrom
naisila/type_formatting

Conversation

@sfc-gh-npuka
Copy link
Copy Markdown
Collaborator

@sfc-gh-npuka sfc-gh-npuka commented Apr 17, 2026

Description

Register pg_lake's custom types in the typedefs list, and combine with the Postgres typedefs for pg_indent. Idea from @sfc-gh-dachristensen #317 (comment)

This PR also fixes our current formatting issues.


Checklist

  • I have tested my changes and added tests if necessary
  • I updated documentation if needed
  • I confirm that all my commits are signed off (DCO)

@sfc-gh-npuka sfc-gh-npuka force-pushed the naisila/type_formatting branch from efd8fda to f74f392 Compare April 17, 2026 13:07
@sfc-gh-npuka sfc-gh-npuka mentioned this pull request Apr 17, 2026
3 tasks
@sfc-gh-npuka sfc-gh-npuka force-pushed the naisila/type_formatting branch from f74f392 to 51a25f3 Compare April 21, 2026 19:33
Signed-off-by: sfc-gh-npuka <naisila.puka@snowflake.com>
Signed-off-by: sfc-gh-npuka <naisila.puka@snowflake.com>
@sfc-gh-npuka sfc-gh-npuka force-pushed the naisila/type_formatting branch from 51a25f3 to 1f695c3 Compare April 21, 2026 19:41
…onIds

The previous grep -v 'struct' filter discarded lines like
"typedef struct CachedExtensionIdsData CachedExtensionIds;" before the
alias pattern could extract CachedExtensionIds. Reformat affected files.

Signed-off-by: sfc-gh-npuka <naisila.puka@snowflake.com>
@sfc-gh-npuka sfc-gh-npuka force-pushed the naisila/type_formatting branch from 1f695c3 to 8d12962 Compare April 21, 2026 19:42
@sfc-gh-npuka sfc-gh-npuka marked this pull request as ready for review April 21, 2026 19:44
@sfc-gh-npuka sfc-gh-npuka changed the title Get pg_lake's custom types registered in the typedefs list for pg_indent Get pg_lake's custom types in the typedefs list for pgindent Apr 21, 2026
@sfc-gh-dachristensen
Copy link
Copy Markdown
Collaborator

I recall some talk of moving to clang-format; is that in the cards? If so, this PR seems like it should probably be closed, but thanks for the work here!

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.

2 participants