Skip to content
CI

Prepare for initial #59

Sign in to view logs

Prepare for initial

Prepare for initial #59

GitHub Actions / solhint completed Nov 20, 2024 in 1s

reviewdog [solhint] report

reported by reviewdog 🐶

Findings (11)

contracts/MallorysMaliciousMisappropriation.sol|5 col 10| imported name FundToken is not used
contracts/NftExchange.sol|5 col 10| imported name Pausable is not used
contracts/NftExchange.sol|6 col 10| imported name Ownable is not used
contracts/NftInvestmentFund.sol|5 col 10| imported name AccessControl is not used
contracts/NftInvestmentFund.sol|25 col 2| Rule is set with explicit type [var/s: uint]
contracts/NftInvestmentFund.sol|80 col 3| GC: Use Custom Errors instead of require statements
contracts/NftInvestmentFund.sol|92 col 4| GC: Use Custom Errors instead of require statements
contracts/NftInvestmentFund.sol|113 col 3| GC: Use Custom Errors instead of require statements
contracts/NftInvestmentFund.sol|134 col 3| GC: Use Custom Errors instead of require statements
contracts/NftInvestmentFund.sol|165 col 3| GC: Use Custom Errors instead of require statements
contracts/NftInvestmentFund.sol|169 col 3| GC: Use Custom Errors instead of require statements

Filtered Findings (0)

Annotations

Check warning on line 5 in contracts/MallorysMaliciousMisappropriation.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/MallorysMaliciousMisappropriation.sol#L5 <no-unused-import> 

imported name FundToken is not used
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":0,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/MallorysMaliciousMisappropriation.sol"},"region":{"startColumn":10,"startLine":5}}}],"message":{"text":"imported name FundToken is not used"},"ruleId":"no-unused-import"}

Check warning on line 5 in contracts/NftExchange.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftExchange.sol#L5 <no-unused-import> 

imported name Pausable is not used
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":1,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftExchange.sol"},"region":{"startColumn":10,"startLine":5}}}],"message":{"text":"imported name Pausable is not used"},"ruleId":"no-unused-import"}

Check warning on line 6 in contracts/NftExchange.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftExchange.sol#L6 <no-unused-import> 

imported name Ownable is not used
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":1,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftExchange.sol"},"region":{"startColumn":10,"startLine":6}}}],"message":{"text":"imported name Ownable is not used"},"ruleId":"no-unused-import"}

Check warning on line 5 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L5 <no-unused-import> 

imported name AccessControl is not used
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":10,"startLine":5}}}],"message":{"text":"imported name AccessControl is not used"},"ruleId":"no-unused-import"}

Check warning on line 25 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L25 <explicit-types> 

Rule is set with explicit type [var/s: uint]
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":2,"startLine":25}}}],"message":{"text":"Rule is set with explicit type [var/s: uint]"},"ruleId":"explicit-types"}

Check warning on line 80 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L80 <gas-custom-errors> 

GC: Use Custom Errors instead of require statements
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":3,"startLine":80}}}],"message":{"text":"GC: Use Custom Errors instead of require statements"},"ruleId":"gas-custom-errors"}

Check warning on line 92 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L92 <gas-custom-errors> 

GC: Use Custom Errors instead of require statements
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":4,"startLine":92}}}],"message":{"text":"GC: Use Custom Errors instead of require statements"},"ruleId":"gas-custom-errors"}

Check warning on line 113 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L113 <gas-custom-errors> 

GC: Use Custom Errors instead of require statements
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":3,"startLine":113}}}],"message":{"text":"GC: Use Custom Errors instead of require statements"},"ruleId":"gas-custom-errors"}

Check warning on line 134 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L134 <gas-custom-errors> 

GC: Use Custom Errors instead of require statements
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":3,"startLine":134}}}],"message":{"text":"GC: Use Custom Errors instead of require statements"},"ruleId":"gas-custom-errors"}

Check warning on line 165 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L165 <gas-custom-errors> 

GC: Use Custom Errors instead of require statements
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":3,"startLine":165}}}],"message":{"text":"GC: Use Custom Errors instead of require statements"},"ruleId":"gas-custom-errors"}

Check warning on line 169 in contracts/NftInvestmentFund.sol

See this annotation in the file changed.

@github-actions github-actions / solhint

[solhint] contracts/NftInvestmentFund.sol#L169 <gas-custom-errors> 

GC: Use Custom Errors instead of require statements
Raw output 
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"index":2,"uri":"file:///home/runner/work/sme4dd-eth-lab2/sme4dd-eth-lab2/contracts/NftInvestmentFund.sol"},"region":{"startColumn":3,"startLine":169}}}],"message":{"text":"GC: Use Custom Errors instead of require statements"},"ruleId":"gas-custom-errors"}
View more details on GitHub Actions