crytic / crytic/slither

Three different separation symbols in config file should be changed to one

Open
#2,045 2 comments 0 reactions 0 assignees View on GitHub
enhancement ux
Dominant language
Python
Stars
6.4k
Forks
1.1k
PR merge metrics
No merged PRs in 30d

Description

### Describe the desired feature

The config file `slither.config.json` confusingly uses three different symbols to separate arguments.

```
{
"detectors_to_exclude": "solc-version,similar-names,timestamp,dead-code",
...
"filter_paths": "@openzeppelin/contracts|contracts/itf|contracts/mocks",
...
"solc_remaps": "@openzeppelin=node_modules/@openzeppelin abdk-libraries-solidity=node_modules/abdk-libraries-solidity"
}
```

- ',' (comma)
- '|' (bar)
- ' ' (space)

Using one specific symbol can save some unpleasant debugging time.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.