Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix test name typo #12

Merged
merged 4 commits into from
Oct 11, 2023
Merged

Fix test name typo #12

merged 4 commits into from
Oct 11, 2023

Conversation

heliuchuan
Copy link
Contributor

Description

Test Plan

Related Links

@heliuchuan heliuchuan requested a review from a team as a code owner October 11, 2023 06:00
@heliuchuan heliuchuan enabled auto-merge (squash) October 11, 2023 06:00
@xbtmatt
Copy link
Contributor

xbtmatt commented Oct 11, 2023

Let's fix the other 3 in here too, and we should probably in general try to do all (or at least a lot of) typos at once :p

aptos-ts-sdk/tests/e2e/api/transaction_submission.test.ts

Line 42 in 300d862

const desirializedTransaction = RawTransaction.deserialize(deserializer);

@gregnazario gregnazario disabled auto-merge October 11, 2023 13:55
@heliuchuan heliuchuan merged commit a2cdde8 into main Oct 11, 2023
5 checks passed
@heliuchuan heliuchuan deleted the heliuchuan-patch-1 branch October 11, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xbtmatt xbtmatt xbtmatt left review comments

@gregnazario gregnazario gregnazario approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@heliuchuan @xbtmatt @gregnazario