Skip to content

Conversation

rarquevaux
Copy link
Contributor

@rarquevaux rarquevaux commented Sep 22, 2025

Explanation

The goal of this PR is to make sure the automatic gas fee update is disabled when isAutomaticGasFeeUpdateEnabled returns false.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

Note

Adds a unit test verifying updateTransactionGasFees skips auto gas estimate updates when isAutomaticGasFeeUpdateEnabled returns false while still applying provided gas params.

Written by Cursor Bugbot for commit 0de2ec6. This will update automatically on new commits. Configure here.

@rarquevaux rarquevaux requested a review from a team as a code owner September 22, 2025 12:18
@rarquevaux rarquevaux force-pushed the feat/STX-169-coverage branch from f72d76f to caa8d58 Compare September 22, 2025 12:20
cursor[bot]

This comment was marked as outdated.

@rarquevaux rarquevaux force-pushed the feat/STX-169-coverage branch 2 times, most recently from a95ba10 to 5392fd7 Compare September 22, 2025 12:43
@rarquevaux rarquevaux force-pushed the feat/STX-169-coverage branch from 5392fd7 to c5f8807 Compare September 23, 2025 14:18
@OGPoyraz OGPoyraz self-requested a review September 25, 2025 08:10
@rarquevaux rarquevaux enabled auto-merge (squash) October 13, 2025 17:17
@rarquevaux rarquevaux force-pushed the feat/STX-169-coverage branch from c5f8807 to 0de2ec6 Compare October 13, 2025 17:17
@rarquevaux rarquevaux merged commit d977d84 into main Oct 13, 2025
243 checks passed
@rarquevaux rarquevaux deleted the feat/STX-169-coverage branch October 13, 2025 17:23
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.

3 participants