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.

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 for --shutdown-on-slot-synced #1114

Open
dorin100 opened this issue Jun 1, 2022 · 2 comments
Open

test for --shutdown-on-slot-synced #1114

dorin100 opened this issue Jun 1, 2022 · 2 comments
Labels

Comments

@dorin100
Copy link
Collaborator

dorin100 commented Jun 1, 2022

details here - IntersectMBO/cardano-node#3670

@dorin100 dorin100 added the node label Jun 1, 2022
@saratomaz saratomaz self-assigned this Jun 1, 2022
@mkoura
Copy link
Collaborator

mkoura commented Jun 1, 2022

Who is using this functionality? IMHO this very low priority test, if it is worth implementing at all :-) It would be exactly the same test is already implemented in cardano-node.

@saratomaz saratomaz removed their assignment Jun 1, 2022
@dorin100
Copy link
Collaborator Author

dorin100 commented Jun 2, 2022

I don't expect it to be a widely used functionality but like anything we make publicly available, it might and will be used by the community too, so having a test for it in our regression would make sense.

I agree that compared with the other tasks in our backlog this has a lower priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants