Skip to content

Update HIP-1193 to reflect current approach#1417

Merged
mgarbs merged 8 commits intomainfrom
hip-1193-rework
Mar 18, 2026
Merged

Update HIP-1193 to reflect current approach#1417
mgarbs merged 8 commits intomainfrom
hip-1193-rework

Conversation

@steven-sheehy
Copy link
Contributor

Description:

Update HIP-1193 to reflect current approach

Related issue(s):

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
@steven-sheehy steven-sheehy self-assigned this Mar 4, 2026
Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
@jsync-swirlds jsync-swirlds self-requested a review March 11, 2026 15:41
Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
Copy link
Contributor

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

Good additions. 1st pass comments

Co-authored-by: Joseph S. <121976561+jsync-swirlds@users.noreply.github.com>
Signed-off-by: Mark Blackman <140427262+Mark-Swirlds@users.noreply.github.com>
Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
@steven-sheehy steven-sheehy requested a review from Nana-EC March 17, 2026 16:45
@steven-sheehy steven-sheehy marked this pull request as ready for review March 17, 2026 20:42
@steven-sheehy steven-sheehy requested a review from a team as a code owner March 17, 2026 20:42
@mgarbs mgarbs merged commit 69dc2b3 into main Mar 18, 2026
10 of 14 checks passed
@mgarbs mgarbs deleted the hip-1193-rework branch March 18, 2026 14:57
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.

6 participants