-
Notifications
You must be signed in to change notification settings - Fork 40
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
improve the readme for new users. #191
base: main
Are you sure you want to change the base?
Conversation
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.
Hi Anja, thanks for the contribution. I have added some comments.
While I think there is a lot of room of improvement in the README, this is better than what we had.
README.md
Outdated
|
||
```sh | ||
nix build | ||
``` | ||
|
||
# Developing | ||
|
||
Enter a nix-shell to get the dependencies of the project: | ||
After building, enter a nix-shell to get the dependencies of the project: |
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.
You don't have to build to enter a nix-shell
README.md
Outdated
|
||
```sh | ||
nix develop | ||
``` | ||
|
||
you can then use `cabal` as normal: | ||
you can then use `cabal` as normal from where you have `act.cabal`, could be in `./src`: |
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.
act.cabal
is at the root directory of act and it won't move (unless we move it).
``` | ||
|
||
To update the project dependencies run: | ||
|
||
```sh | ||
nix flake update | ||
``` | ||
|
||
# Usage |
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.
There is also a --help
option for each one of them that describes all the available command line options.
README.md
Outdated
## SMT | ||
|
||
```sh | ||
act prove --file <PATH_TO_SPEC> |
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.
Being able to use the act
executable relies on having run make
first, as it is the Makefile that runs the command to build it (of course one could also run it by hand).
I suggest using cabal run act
instead which has no prerequisites, or specifying that make
has to be run first.
The readme and the docs are now updated according to Zoe's suggestions. |
I had some issues at first to set up act on my machine, so I updated the readme a bit to match the current state of the repository.