Skip to content

adding linting to submodules #16

adding linting to submodules

adding linting to submodules #16

Triggered via push February 16, 2024 06:45
Status Failure
Total duration 1m 51s
Artifacts

ci.yaml

on: push
test-unit  /  forge-test
13s
test-unit / forge-test
test-safe7579  /  forge-test
23s
test-safe7579 / forge-test
test-sessionkeymanager  /  forge-test
14s
test-sessionkeymanager / forge-test
test-modulekit-examples  /  forge-test
24s
test-modulekit-examples / forge-test
lint-safe7579  /  forge-test
12s
lint-safe7579 / forge-test
lint-sessionkeymanager  /  forge-test
24s
lint-sessionkeymanager / forge-test
lint-modulekit-examples  /  forge-test
16s
lint-modulekit-examples / forge-test
Fit to window
Zoom out
Zoom in

Annotations

11 errors and 45 warnings
lint-modulekit-examples / forge-test
Unexpected import of console file
lint-modulekit-examples / forge-test
Unexpected console statement
lint-modulekit-examples / forge-test
Unexpected import of console file
lint-modulekit-examples / forge-test
Process completed with exit code 1.
lint-sessionkeymanager / forge-test
Line length must be no more than 123 but current length is 6718
lint-sessionkeymanager / forge-test
Process completed with exit code 1.
lint-safe7579 / forge-test
Unexpected import of console file
lint-safe7579 / forge-test
Compiler version >=0.7.0 <0.9.0 does not satisfy the >=0.8.0 semver requirement
lint-safe7579 / forge-test
Unexpected import of console file
lint-safe7579 / forge-test
Compiler version must be declared
lint-safe7579 / forge-test
Process completed with exit code 1.
lint / forge-lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint / forge-lint: src/core/ExtensibleFallbackHandler.sol#L40
Explicitly mark visibility of state
lint / forge-lint: src/core/ExtensibleFallbackHandler.sol#L124
Variable "smartAccount" is unused
lint / forge-lint: src/deployment/RegistryDeployer.sol#L31
Explicitly mark visibility of state
lint / forge-lint: src/deployment/RegistryDeployer.sol#L33
Explicitly mark visibility of state
lint / forge-lint: src/integrations/ERC20.sol#L5
imported name IERC7579Account is not used
lint / forge-lint: src/integrations/ERC4626.sol#L5
imported name IERC7579Account is not used
lint / forge-lint: src/integrations/ERC721.sol#L5
imported name IERC7579Account is not used
lint / forge-lint: src/integrations/uniswap/MockUniswap.sol#L4
global import of path ../interfaces/uniswap/v3/ISwapRouter.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint / forge-lint: src/integrations/uniswap/v3/Uniswap.sol#L8
imported name IERC7579Account is not used
lint / forge-lint: src/mocks/MockExecutor.sol#L5
imported name IERC7579Account is not used
build / forge-build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/save@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
test-unit / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
test-sessionkeymanager / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint-modulekit-examples / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint-modulekit-examples / forge-test
imported name ModuleTypeLib is not used
lint-modulekit-examples / forge-test
imported name ModuleType is not used
lint-modulekit-examples / forge-test
global import of path forge-std/console2.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
global import of path forge-std/Test.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
global import of path @rhinestone/modulekit/src/ModuleKit.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
global import of path @rhinestone/modulekit/src/Helpers.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
global import of path @rhinestone/modulekit/src/Core.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
global import of path solmate/test/utils/mocks/MockERC20.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
global import of path solmate/test/utils/mocks/MockERC4626.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-modulekit-examples / forge-test
imported name MODULE_TYPE_VALIDATOR is not used
test-modulekit-examples / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
test-safe7579 / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint-sessionkeymanager / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint-sessionkeymanager / forge-test
imported name IERC7579Account is not used
lint-sessionkeymanager / forge-test
imported name ModuleTypeLib is not used
lint-sessionkeymanager / forge-test
imported name ModuleType is not used
lint-sessionkeymanager / forge-test
imported name IERC1271 is not used
lint-sessionkeymanager / forge-test
Variable "userOpHash" is unused
lint-sessionkeymanager / forge-test
Variable "userOpHash" is unused
lint-safe7579 / forge-test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint-safe7579 / forge-test
global import of path forge-std/Test.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-safe7579 / forge-test
global import of path @safe-global/safe-contracts/contracts/Safe.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-safe7579 / forge-test
Explicitly mark visibility of state
lint-safe7579 / forge-test
Explicitly mark visibility of state
lint-safe7579 / forge-test
Explicitly mark visibility of state
lint-safe7579 / forge-test
global import of path ./interfaces/ISafe.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-safe7579 / forge-test
global import of path forge-std/console2.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-safe7579 / forge-test
global import of path ./ModuleManager.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-safe7579 / forge-test
global import of path erc7579/interfaces/IERC7579Account.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
lint-safe7579 / forge-test
global import of path erc7579/interfaces/IERC7579Module.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)