Skip to content

Apply PR suggestions #168

Apply PR suggestions

Apply PR suggestions #168

Triggered via push August 20, 2024 12:40
Status Failure
Total duration 7m 19s
Artifacts

main.yaml

on: push
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 11 warnings
linting: contracts/contracts/coordination/InfractionCollector.sol#L27
Replace ⏎············address(_coordinator)·!=·address(0),⏎············"Contracts·must·be·specified"⏎········ with address(_coordinator)·!=·address(0),·"Contracts·must·be·specified"
linting: contracts/contracts/coordination/InfractionCollector.sol#L53
Replace InfractionType.MISSING_TRANSCRIPT with ⏎····················InfractionType.MISSING_TRANSCRIPT⏎················
linting
Process completed with exit code 1.
linting: contracts/contracts/Adjudicator.sol#L31
Function order is incorrect, state variable declaration can not go after event definition (line 25)
linting: contracts/contracts/Adjudicator.sol#L37
Main key parameter in mapping penaltyHistory is not named
linting: contracts/contracts/Adjudicator.sol#L37
Value parameter in mapping penaltyHistory is not named
linting: contracts/contracts/Adjudicator.sol#L38
Main key parameter in mapping evaluatedCFrags is not named
linting: contracts/contracts/Adjudicator.sol#L38
Value parameter in mapping evaluatedCFrags is not named
linting: contracts/contracts/Adjudicator.sol#L53
Use Custom Errors instead of require statements
linting: contracts/contracts/Adjudicator.sol#L90
Use Custom Errors instead of require statements
linting: contracts/contracts/Adjudicator.sol#L102
Use Custom Errors instead of require statements
linting: contracts/contracts/Adjudicator.sol#L106
Use Custom Errors instead of require statements
linting: contracts/contracts/Adjudicator.sol#L114
Use Custom Errors instead of require statements
tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3, actions/setup-python@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/