Added support for ALTER OPERATOR CLASS syntax
#2135
Conversation
| })) | ||
| } | ||
|
|
||
| pub fn parse_alter_operator_class(&mut self) -> Result<Statement, ParserError> { |
There was a problem hiding this comment.
Can we add documentation for the function?
There was a problem hiding this comment.
Can I add the deny missing doc to clippy?
There was a problem hiding this comment.
I have added the deny missing doc, but there is an impressive amount of missing doc. I am trying to add the missing documentation so to make it impossible for missing documentation in future development.
There was a problem hiding this comment.
Ok, it was quite an effort but now #![forbid(missing_docs)] is in place and everything public is documented
iffyio
left a comment
iffyio
left a comment
There was a problem hiding this comment.
LGTM! Thanks @LucaCappelletti94!
|
@LucaCappelletti94 could you look to resolve the conflicts on this branch when you have some time? |
|
I fixed stuff rapidly in the github editor, so likely there will be some mess. Will fix in a sec. |
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
|
I have documented the code that had no documentation in the main. |
2 participants
Added support for
ALTER OPERATOR CLASSsyntax.