DO NOT MERGE - chore: Cleanup and improvements (underscore on private/internal fields) #388
alainncls
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
11 warnings
|
lint:
src/ModuleRegistry.sol#L133
Avoid to use tx.origin
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L20
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L21
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L45
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L46
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L51
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L52
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L65
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/AttestationRegistryMock.sol#L70
Use Custom Errors instead of require statements
|
|
lint:
test/mocks/ModuleRegistryMock.sol#L24
Use Custom Errors instead of require statements
|
|
coverage
The following actions uses node12 which is deprecated and will be forced to run on node16: terencetcf/github-actions-lcov-minimum-coverage-checker@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|