Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Rename transaction v2 proposal #28508

Merged
merged 1 commit into from
Oct 21, 2022

Conversation

jstarry
Copy link
Contributor

@jstarry jstarry commented Oct 20, 2022

Problem

It's confusing that the address lookup tables / versioned transactions proposal uses v2 instead of v0 when referring to the new transaction version.

Summary of Changes

  • Rename the versioned transactions proposal title
  • Cleaned up outdated parts of the proposal

Fixes #

@jstarry jstarry requested a review from jordaaash October 20, 2022 15:51
Copy link
Contributor

@jordaaash jordaaash left a comment

Choose a reason for hiding this comment

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

Looks really good!

@jstarry jstarry added the automerge Merge this Pull Request automatically once CI passes label Oct 20, 2022
@mergify mergify bot removed the automerge Merge this Pull Request automatically once CI passes label Oct 20, 2022
@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2022

automerge label removed due to a CI failure

@jstarry jstarry merged commit 199398d into solana-labs:master Oct 21, 2022
@jstarry jstarry deleted the proposal/versioned-transactions branch October 21, 2022 01:29
gnapoli23 pushed a commit to gnapoli23/solana that referenced this pull request Dec 16, 2022
Rename transactions v2 proposal to clear up versioning
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants