feat: As an attestation issuer, I would like to remain compatible with the ERC-721 standard #390
Triggered via pull request
September 21, 2023 11:32
satyajeetkolhapure
Status
Failure
Total duration
2m 24s
Artifacts
–
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
1 error and 11 warnings
|
coverage
The current code coverage (92%) is below the minimum 96% threshold.
|
|
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/
|