Merge pull request #30 from bcnmy/feat/refactor-events-and-tests #158
Annotations
11 warnings
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, pnpm/action-setup@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L7
global import of path @openzeppelin/contracts/utils/introspection/IERC165.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L10
global import of path account-abstraction/core/UserOperationLib.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L18
Immutable variables name are set to be in capitalized SNAKE_CASE
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L109
GC: Use Custom Errors instead of require statements
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L156
GC: Use Custom Errors instead of revert statements
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L163
GC: Use Custom Errors instead of require statements
|
|
Lint sources:
contracts/base/BasePaymaster.sol#L171
Avoid to use inline assembly. It is acceptable only in rare cases
|
|
Lint sources:
contracts/interfaces/IBiconomySponsorshipPaymaster.sol#L4
global import of path @openzeppelin/contracts/token/ERC20/IERC20.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
|
|
Lint sources:
contracts/interfaces/IBiconomySponsorshipPaymaster.sol#L12
'_paymasterId' should not start with _
|
|
Lint sources:
contracts/interfaces/IBiconomySponsorshipPaymaster.sol#L12
'_value' should not start with _
|
Loading