Skip to content
lint

Combine approve, deposit, stake/unstake into a single method #1587

Sign in to view logs

Combine approve, deposit, stake/unstake into a single method

Combine approve, deposit, stake/unstake into a single method #1587

Triggered via pull request October 17, 2024 08:03
@kevin-fruitfulkevin-fruitful
synchronize #150
combine-approve-deposit-stake
Status Success
Total duration 37s
Artifacts –

lint.yml

on: pull_request
Lint contracts
26s
Lint contracts
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
Lint contracts
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Lint contracts: src/facets/ACLFacet.sol#L27
Use Custom Errors instead of require statements
Lint contracts: src/facets/ACLFacet.sol#L48
Use Custom Errors instead of require statements
Lint contracts: src/facets/ACLFacet.sol#L149
Use Custom Errors instead of require statements
Lint contracts: src/facets/GovernanceFacet.sol#L46
Use Custom Errors instead of revert statements
Lint contracts: src/facets/GovernanceFacet.sol#L64
Use Custom Errors instead of require statements
Lint contracts: src/facets/GovernanceFacet.sol#L78
Use Custom Errors instead of require statements
Lint contracts: src/facets/MarketFacet.sol#L36
Use Custom Errors instead of require statements
Lint contracts: src/facets/MarketFacet.sol#L38
Use Custom Errors instead of require statements
Lint contracts: src/facets/NaymsOwnershipFacet.sol#L17
Use Custom Errors instead of require statements
Lint contracts: src/facets/NaymsOwnershipFacet.sol#L18
Use Custom Errors instead of require statements