Skip to content

Conversation

n-r-w
Copy link
Contributor

@n-r-w n-r-w commented Jan 12, 2024

For postgresql, comments for tables are not extracted from the database. In addition, the default template does not use information about comments.

@kenshaw
Copy link
Member

kenshaw commented Jan 12, 2024

@n-r-w This is fine, could you rebase this as a single commit though? Also, if you wanted to add comments for the PostgreSQL schemas in the a_bit_of_everything or the booktest examples, it would be helpful.

@n-r-w n-r-w force-pushed the fix_psql_comments branch from 90d36f1 to c001e59 Compare January 12, 2024 20:48
@n-r-w
Copy link
Contributor Author

n-r-w commented Jan 12, 2024

@n-r-w This is fine, could you rebase this as a single commit though? Also, if you wanted to add comments for the PostgreSQL schemas in the a_bit_of_everything or the booktest examples, it would be helpful.

Rebase is completed.
Regarding the examples of a_bit_of_everything and book test, it is not very clear where to start there to add comments to the database schema.

@kenshaw
Copy link
Member

kenshaw commented Jan 12, 2024

@n-r-w thanks for the rebase. Will merge this. Just was hoping you'd save me a bit of work on the comments/schema example ;)

@kenshaw kenshaw merged commit ce2cf40 into xo:master Jan 12, 2024
@n-r-w n-r-w deleted the fix_psql_comments branch January 12, 2024 21:48
kenshaw added a commit to xo/homebrew-xo that referenced this pull request May 24, 2024
## What's Changed
* fix psql comments by @n-r-w in xo/dbtpl#404
* Fix postresql multiline comments by @n-r-w in xo/dbtpl#406

## New Contributors
* @n-r-w made their first contribution in xo/dbtpl#404

**Full Changelog**: xo/dbtpl@v1.0.1...v1.0.2
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