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: remove base sepolia url from env vars #162

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

joepegler
Copy link
Collaborator

@joepegler joepegler commented Jan 6, 2025

PR-Codex overview

This PR focuses on updating the version of the @biconomy/sdk, making changes to environment variables, and modifying the deployment logic in the codebase.

Detailed summary

  • Updated version in package.json from 0.0.23 to 0.0.24.
  • Removed tenderlyUrl from environment variables in CHANGELOG.md.
  • Introduced BASE_SEPOLIA_RPC_URL in src/test/testUtils.ts.
  • Updated the forkUrl logic in toConfiguredAnvil.
  • Enhanced initDeployments function to accept shouldForkBaseSepolia.
  • Conditional logic added for bytecode deployment based on shouldForkBaseSepolia.
  • Adjusted index value in multiple test cases in src/sdk/modules/smartSessionsValidator/toSmartSessionValidator.enable.mode.test.ts.
  • Increased preVerificationGas from 10000000n to 100000000n in user operation preparation.

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

Copy link

github-actions bot commented Jan 6, 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 force-pushed the chore/remove_tenderly_url_from_env_vars branch from 3105090 to 829ddfd Compare January 6, 2025 13:21
@joepegler joepegler merged commit f6950b4 into develop Jan 6, 2025
4 checks passed
@joepegler joepegler deleted the chore/remove_tenderly_url_from_env_vars branch January 6, 2025 13:22
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