Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Command line enhancements #112

Open
wants to merge 100 commits into
base: main
Choose a base branch
from
Open

Command line enhancements #112

wants to merge 100 commits into from

Conversation

lmcmicu
Copy link
Collaborator

@lmcmicu lmcmicu commented Nov 5, 2024

Closes #106

@lmcmicu
add add subcommand
0734540
@lmcmicu
read input from stdin in Add
9eef094
@lmcmicu
use nested subcommands
c32574f
@lmcmicu
add get subcommand, alphabetize match arms in main(), add get_row() stub
e2038fb
@lmcmicu
move all positional arguments to leaf subcommands
d160339
@lmcmicu
reorganize toolkit
a73e21a
@lmcmicu
implement API version of get_row()
4b89c58
@lmcmicu
rename get_row() to get_row_from_db() and get_row_from_db() to get_ro…
9126d61 
…w_from_db_tx()
@lmcmicu
use AnyKind instead of AnyPool where possible
d0002e8 
doing this is useful to help to distinguish those functions that
actually require database access and those that don't
@lmcmicu
add DbKind
fadfa65
@lmcmicu
small reorg and name change for functions that use transactions
ce528e2
@lmcmicu
add 'get cell' and 'get value' subcommands
5fab3e4
@lmcmicu
correct SerdeValues in get_row_from_db()
65de31c
@lmcmicu
implement get table
89ad162
@lmcmicu
get source and database from environment if not specified
ce48032
@lmcmicu
implement get messages
7ac4568
@lmcmicu
implement CLI validate
93884df
@lmcmicu
export environment vars
9b2e520
@lmcmicu
fix tests
266301e
@lmcmicu
use simple json for input rows
536585b
@lmcmicu
implement CLI update and delete
c2ababb
@lmcmicu
add subcommands for update and delete
2bf060c
@lmcmicu
make row and column optional when getting messages
885d2cb
@lmcmicu
add add message
10fbf70
@lmcmicu
add update message
b293aa4
@lmcmicu
add delete message
228acc3
@lmcmicu
allow to delete multiple messages
49b70cc
@lmcmicu
add undo redo to cli
28b21db
@lmcmicu
exit with correct status for cli-validate
bf670d3
@lmcmicu
add cli history
78ade3e
lmcmicu added 28 commits December 2, 2024 06:28
@lmcmicu
maybe ask questions when adding messages
0e2a379
@lmcmicu
maybe ask questions when adding rows
c363d30
@lmcmicu
maybe ask questions when updating messages and rows
aba5676
@lmcmicu
refactor validate
d0dd483
@lmcmicu
record column adds to history table
2fa7918
@lmcmicu
refactor
fb92979
@lmcmicu
record column deletes, datatype adds and deletes to history
9ca55e9
@lmcmicu
record renames of tables, datatypes and columns
3bb05a0
@lmcmicu
record table adds and deletes
cc7629d
@lmcmicu
implement undo add, delete column
d7021fd
@lmcmicu
implement undo add, delete datatype
e43ebd0
@lmcmicu
implement undo add, delete table
e701127
@lmcmicu
implement redo metadata ops
f8f25b0
@lmcmicu
WIP tweak
7321261
@lmcmicu
save datatype row number for undo/redo when adding and deleting
fde93f8
@lmcmicu
save table row number when deleting
8872997
@lmcmicu
implement undo, redo for rename_column()
ef5baa9
@lmcmicu
implement undo, redo for rename datatype
32bb4af
@lmcmicu
implement undo and redo for table renames
b2cd68e
@lmcmicu
WIP clean up and refactor
270e336
@lmcmicu
clean up and refactor; rename SerdeMap to JsonRow
cf3d05e
@lmcmicu
preserve row order when undoing datatype, column and table manipulations
ac93a71
@lmcmicu
fix bug in assigning NULL values when inserting a new row
fc0bb4e
@lmcmicu
be more verbose, fix bug in extracting row number from messages
626033d
@lmcmicu
cleanup and refactoring
215e7f4
@lmcmicu
add basic CLI tests
24980c9
@lmcmicu
make references to configuration tables generic where possible
5d11586
@lmcmicu
add missing files
9cd8155
@lmcmicu lmcmicu marked this pull request as ready for review December 23, 2024 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamesaoverton jamesaoverton Awaiting requested review from jamesaoverton

Assignees

@lmcmicu lmcmicu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Prototype CLI for CRUD operations and more
1 participant
@lmcmicu