Skip to content

Conversation

tynes
Copy link
Contributor

@tynes tynes commented Jan 31, 2025

Description

The interop predeploy specs needed some cleanup and standardization.
More to come.

The interop predeploy specs needed some cleanup and standardization.
More to come.
@emhane emhane requested a review from Copilot April 4, 2025 10:37
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

@tynes tynes added the H-interop Hardfork: change planned for Interop upgrade label Apr 7, 2025
@emhane emhane added the C-debt Category: debt label Apr 24, 2025
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@emhane emhane requested review from a team as code owners April 24, 2025 15:00
emhane and others added 2 commits April 24, 2025 17:00
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Member

@emhane emhane left a comment

Choose a reason for hiding this comment

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

can this be rescoped into smaller prs? will avoid merge conflicts over time as approving reviews are pending, and it is more difficult to sign off a large diff as opposed to a small diff in a single review. for smaller prs, there is the chance that first review can approve changes already.

@emhane emhane added the U-smart-contract Upgrade: involving changes to smart contracts label Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-debt Category: debt H-interop Hardfork: change planned for Interop upgrade U-smart-contract Upgrade: involving changes to smart contracts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants