Skip to content
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

Adding remapping keyword to packages flag section #364

Merged
merged 4 commits into from
Feb 25, 2025

Conversation

RedLikeRosesss
Copy link
Contributor

"remapping" isn't mentioned anywhere in the doc. That's why some users got confused by packages flag. Adding a note about remapping is supposed to solve this issue

Copy link
Contributor

@shoham-certora shoham-certora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@@ -403,6 +403,10 @@ By default we look for the packages in `$NODE_PATH`. If there are packages are i
**Example**
`certoraRun Bank.sol --verify Bank:Bank.spec --packages ds-stop=$PWD/lib/ds-token/lib/ds-stop/src ds-note=$PWD/lib/ds-token/lib/ds-stop/lib/ds-note/src`

```{note}
Information about missing packages is usually stored in `remappings.txt` file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider something like:
"In Solidity projects, information about packages' location is usually stored in remappings.txt file."

@RedLikeRosesss RedLikeRosesss merged commit bc4ab05 into master Feb 25, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants