Skip to content

Conversation

@romashka-btc
Copy link

Fix: Replaced !(...) with !== for better readability

Changes

  • Updated the conditional statement to use the !== operator instead of wrapping the condition with !(...).

    • Before:
      if (!(baseTokenAddress === zksync.utils.ETH_ADDRESS_IN_CONTRACTS)) {
      
    • After:
      if (baseTokenAddress !== zksync.utils.ETH_ADDRESS_IN_CONTRACTS) {

Purpose

  • Improved code readability and clarity by using the !== operator instead of a double negative.
  • Addressed the linter suggestion to use the opposite operator (!==) for a cleaner and more maintainable codebase.

Context

  • File: core/tests/revert-test/tests/tester.ts
  • Issue: Linter warning suggesting the replacement of !(...) with !==.

This change ensures better code clarity and follows recommended best practices for logical operations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants