feat: add table-references command group - #22
Conversation
|
Good job looks great! Left a few small comments of things to look into. Please make sure you add a PR description and bump |
ClassicMMT
left a comment
There was a problem hiding this comment.
Looking good. Almost there!
7bea2fb to
792ad95
Compare
ClassicMMT
left a comment
There was a problem hiding this comment.
Approved. Just two last things left to make it look a bit better.
792ad95 to
8435e54
Compare
|
Tests I ran
|
ClassicMMT
left a comment
There was a problem hiding this comment.
Looks good to me. @kw-datamasque all good?
ClassicMMT
left a comment
There was a problem hiding this comment.
Looks good to me. @kw-datamasque all good?
|
Feel free to create a new release once merged in! |
Adds a
table-referencescommand group (list/get/create/update/delete) to dm, mirroring the existingconnectionscommand group, over `datamasque-python TableReferenceClient.createaccepts--file(JSON) or flags for the common case; source is a file path for file connections orschema.tablefor database connections, with format always explicit, never inferred.updatechanges only the fields passed; any CSV/format flag replacesoptionswholesale on the resulting PUT.Bumps the
datamasque-pythonfloor to>=1.2.5,<2.