Skip to content

revert(types): Revert #3543 (backport #3549)#3705

Merged
alesforz merged 4 commits intov0.38.xfrom
alesforz/v0.38.x/revert-pr-3543
Aug 15, 2024
Merged

revert(types): Revert #3543 (backport #3549)#3705
alesforz merged 4 commits intov0.38.xfrom
alesforz/v0.38.x/revert-pr-3543

Conversation

@alesforz
Copy link
Collaborator

@alesforz alesforz commented Aug 14, 2024

Given the discussion in #3528, we are reverting the changes made by #3543 in v0.38.x (backported with #3549).
Additionally, we are removing the changes made in #3543 from the CHANGELOG.


PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
- [] Updated relevant documentation (docs/ or spec/) and code comments

…and `LightClientAttackEvidence` types (backport #3543) (#3549)"

This reverts commit 24b39c5.
@alesforz alesforz self-assigned this Aug 14, 2024
@alesforz alesforz marked this pull request as ready for review August 14, 2024 16:49
@alesforz alesforz requested a review from a team as a code owner August 14, 2024 16:49
@alesforz alesforz requested a review from sergio-mena August 14, 2024 16:49
@alesforz alesforz changed the title Fix(types): Revert #3543 (backport #3549) Revert(types): Revert #3543 (backport #3549) Aug 14, 2024
@sergio-mena sergio-mena changed the title Revert(types): Revert #3543 (backport #3549) revert(types): Revert #3543 (backport #3549) Aug 14, 2024
sergio-mena
sergio-mena previously approved these changes Aug 14, 2024
Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

🙏

@sergio-mena sergio-mena dismissed their stale review August 14, 2024 17:08

realised something last minute

Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

Just one minor comment

Copy link
Collaborator

@melekes melekes left a comment

Choose a reason for hiding this comment

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

👍

…se the CHANGELOG

will update automatically with the next point release.

This reverts commit ece7c8e.
@alesforz alesforz merged commit b55fa31 into v0.38.x Aug 15, 2024
@alesforz alesforz deleted the alesforz/v0.38.x/revert-pr-3543 branch August 15, 2024 07:09
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