-
Notifications
You must be signed in to change notification settings - Fork 979
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add instruction for how to upgrade slither in the CLI #2496
Conversation
WalkthroughWalkthroughThe changes to the project primarily involve updates to the README.md file with instructions on upgrading the Slither version. These updates aim to ensure users have access to the latest features and bug fixes associated with the tool. There are no changes to the declarations of exported or public entities in the codebase. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- README.md (1 hunks)
Additional context used
LanguageTool
README.md
[uncategorized] ~45-~45: Do not mix variants of the same word (‘analyse’ and ‘analyze’) within a single text. (EN_WORD_COHERENCY)
Context: ...lithIR)) enables simple, high-precision analyses * Correctly parses 99.9% of all public ...
[uncategorized] ~193-~193: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...tional | High 76 |low-level-calls
| [Low level calls](https://github.com/crytic/slithe...
[uncategorized] ~222-~222: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...Quick Review Printers *human-summary
: [Print a human-readable summary of the ...
[uncategorized] ~229-~229: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...In-Depth Review Printers *call-graph
: [Export the call-graph of the contracts...
[grammar] ~230-~230: The noun should probably be in the singular form. (EVERY_EACH_SINGULAR)
Context: ...graph) *cfg
: [Export the CFG of each functions](https://github.com/trailofbits/slither...
[uncategorized] ~241-~241: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ... Tools *slither-check-upgradeability
: [Reviewdelegatecall
-based upgradeabi...
[style] ~258-~258: The phrase ‘feel free to’ is used quite frequently. Consider using a less frequent alternative to set your writing apart from others and make it sound more professional. (FEEL_FREE_TO_STYLE_ME)
Context: ...lither/slither.html). ## Getting Help Feel free to stop by our [Slack channel](https://emp...
[uncategorized] ~263-~263: Do not mix variants of the same word (‘analyse’ and ‘analyze’) within a single text. (EN_WORD_COHERENCY)
Context: ...cribes how to write a new vulnerability analyses. * The [API documentation](https://git...
[misspelling] ~304-~304: Possible spelling mistake found. (EN_MULTITOKEN_SPELLING_TWO)
Context: ...detect loops | Chunmiao Li, Shijie Nie, Yang Cao, Yijun Yu, Zhenjiang Hu | IEEE Open J. ...
[typographical] ~307-~307: Two consecutive dots (DOUBLE_PUNCTUATION)
Context: ...t tokens' features (mintable, pausable, ..) | Mazorra, Bruno, Victor Adan, and Van...
[typographical] ~312-~312: Two consecutive dots (DOUBLE_PUNCTUATION)
Context: ...tures (CFG, function, parameters types, ..) | Qing Huang, Dianshu Liao, Zhenchang ...
[duplication] ~313-~313: Possible typo: you repeated a word (ENGLISH_WORD_REPEAT_RULE)
Context: ...uild state access graphs | Xiaodong Qi, Jiao Jiao, Yi Li | International Conference on Di...
Additional comments not posted (1)
README.md (1)
79-83
: Clear and Useful Upgrade InstructionsThe added instructions for upgrading Slither using pip are clear and effectively communicate the necessary steps to keep the tool updated. This is a valuable addition for users who need to stay current with the latest features and bug fixes.
Closing as did not checked out from |
Just added a small instruction under the "Install with Git" part, so that developer know how to easily upgrade Slither in their CLI to keep up with the latest features / bug fixes releases.
Summary by CodeRabbit