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

Add threshold custody commands to pcli #3449

Merged
merged 13 commits into from
Dec 2, 2023
Merged

Conversation

cronokirby
Copy link
Contributor

@cronokirby cronokirby commented Nov 29, 2023

Closes #3343.
Closes #3344.
Closes #3345.

Probably want to merge #3383 first, then switch this PR to be based on main, and review it.

@cronokirby cronokirby force-pushed the 3342-threshold-custody-backend branch from e4e1784 to fa0d488 Compare November 29, 2023 19:00
@cronokirby cronokirby force-pushed the threshold-custody-pcli branch from 39be315 to b4de742 Compare November 29, 2023 19:04
Base automatically changed from 3342-threshold-custody-backend to main November 30, 2023 00:07
Co-authored-by: redshiftzero <[email protected]>
Signed-off-by: Lúcás Meier <[email protected]>
@cronokirby
Copy link
Contributor Author

@hdevalence
Copy link
Member

hdevalence commented Dec 1, 2023

I don't know, it doesn't seem obviously triggered by a failing expectation, we can re-run the job and find out.

I'm noticing, however, that the smoke test job is taking alarmingly long to run. It should be finishing within 5-10 minutes, not up to 30. EDIT: I misread the timestamps, it takes about 20 minutes. (This is still longer than ideal).

@cronokirby
Copy link
Contributor Author

Well, seems like it worked out now.

@cronokirby cronokirby merged commit 0e2b51c into main Dec 2, 2023
5 checks passed
@cronokirby cronokirby deleted the threshold-custody-pcli branch December 2, 2023 00:03
@conorsch conorsch mentioned this pull request Feb 8, 2024
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants