-
Notifications
You must be signed in to change notification settings - Fork 156
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.
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
Test restarting db-sync with alternate command line params #1466
Conversation
209cc82
to
00fd409
Compare
4f3b562
to
950b92a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
1b55527
to
8417f1d
Compare
81c1e5b
to
8a37938
Compare
cardano-chain-gen/test/Test/Cardano/Db/Mock/Unit/Babbage/CommandLineArg/ForceIndex.hs
Outdated
Show resolved
Hide resolved
cardano-chain-gen/test/Test/Cardano/Db/Mock/Unit/Babbage/CommandLineArg/ForceIndex.hs
Outdated
Show resolved
Hide resolved
cardano-db/src/Cardano/Db/Migration/Extra/CosnumedTxOut/Queries.hs
Outdated
Show resolved
Hide resolved
3a7326e
to
c5e8706
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
The single comment I left is actually already tracked by #1489
wasPruneTxOutPreviouslySet = elem PruneTxOutFlagPreviouslySet | ||
|
||
data PruneConsumeMigration = PruneConsumeMigration | ||
{ pcmConsume :: Bool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this ever used? Maybe it can be removed
Description
This fixes:
#1465
#1486
Checklist
fourmolu
on version 0.10.1.0 (which can be run withscripts/fourmolize.sh
)Migrations
If there is a breaking change, especially a big one, please add a justification here. Please elaborate
more what the migration achieves, what it cannot achieve or why a migration is not possible.