Skip to content

Conversation

@kwvg
Copy link
Collaborator

@kwvg kwvg commented Mar 25, 2022

No description provided.

@kwvg kwvg changed the title merge bitcoin#14794..17367: ci reworking merge bitcoin#14794..17367: ci reworking (part 2) Mar 25, 2022
@kwvg kwvg force-pushed the ci2 branch 2 times, most recently from 8a27475 to c4669b8 Compare March 25, 2022 10:07
@github-actions
Copy link

This pull request has conflicts, please rebase.

@kwvg kwvg force-pushed the ci2 branch 2 times, most recently from 1a7ade4 to f395ad7 Compare June 22, 2022 17:22
@kwvg kwvg marked this pull request as ready for review June 22, 2022 17:23
@kwvg kwvg requested review from PastaPastaPasta and UdjinM6 June 22, 2022 18:11
@kwvg kwvg changed the title merge bitcoin#14794..17367: ci reworking (part 2) backport: bitcoin#16597, #17176, #14794, #17205, #17240, #17367, #18862, #21405 (ci reworking: part 2) Jun 22, 2022
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK for merging via merge commit

@UdjinM6 UdjinM6 added this to the 18.1 milestone Jun 24, 2022
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! 👍 but ci is a bit broken now :) I think UdjinM6@da878b1 should fix it (waiting for https://gitlab.com/UdjinM6/dash/-/pipelines/572355610 to confirm)

UdjinM6
UdjinM6 previously approved these changes Jun 25, 2022
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@github-actions
Copy link

This pull request has conflicts, please rebase.

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, utACK

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK for merging via merge commit

@UdjinM6 UdjinM6 merged commit ef8cf4b into dashpay:develop Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants