Skip to content

Conversation

@michaeldiamant
Copy link

For illustrative purposes, abstract over list of application creation to minimize code duplication.

Notes:

  • As a way to boost my Python learning while reviewing on Tests: E2E testing C2C with ClearStateProgram algorand/go-algorand#3693, I replaced imperative application creation calls with a helper method.
  • The PR's minor novelty is using more-itertools to abstract over partitioning a list of errors and transaction infos.
  • Since the imperative logic is also relatively straightforward, I don't expect to have the PR merged. I'm posting it in case there's other folks new to Python that can benefit from seeing the before/after transformation. Though if we wish to merge it, I'm open to feedback.

@michaeldiamant michaeldiamant deleted the e2e_csp_partition_example branch March 3, 2022 19:25
algoidurovic pushed a commit that referenced this pull request May 6, 2022
algoidurovic pushed a commit that referenced this pull request Sep 8, 2022
Tests: Remove using unreleased semicolon support in AVM test
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.

1 participant