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.

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

chore: index test #163

Closed
wants to merge 1 commit into from
Closed

chore: index test #163

wants to merge 1 commit into from

Conversation

joepegler
Copy link
Collaborator

@joepegler joepegler commented Jan 7, 2025

PR-Codex overview

This PR updates the version of the @biconomy/sdk, removes the tenderlyUrl from environment variables, and modifies various functions to improve deployment and testing processes, including adjustments to RPC URLs and deployment parameters.

Detailed summary

  • Updated version in package.json from 0.0.23 to 0.0.24.
  • Removed tenderlyUrl from environment variables in CHANGELOG.md.
  • Added BASE_SEPOLIA_RPC_URL to src/test/testUtils.ts.
  • Changed forkUrl logic in toConfiguredAnvil function.
  • Updated initDeployments function to accept shouldForkBaseSepolia parameter.
  • Adjusted logging and deployment conditions in initDeployments.
  • Replaced hardcoded index values with a variable index in test files.
  • Increased preVerificationGas limit in several tests.
  • Modified test descriptions to conditionally run based on paymasterTruthy.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@joepegler joepegler changed the base branch from develop to main January 7, 2025 12:38
@joepegler joepegler changed the base branch from main to develop January 7, 2025 12:38
Copy link

github-actions bot commented Jan 7, 2025

size-limit report 📦

Path Size
core (esm) 16.32 KB (0%)
core (cjs) 22.77 KB (0%)
bundler (tree-shaking) 518 B (0%)
paymaster (tree-shaking) 543 B (0%)

@joepegler joepegler closed this Jan 7, 2025
@joepegler joepegler deleted the feat/index_test branch January 7, 2025 12:40
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