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

improve the readme for new users. #191

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

anjapetkovic
Copy link

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.

Copy link
Collaborator

@zoep zoep left a 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:
Copy link
Collaborator

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`:
Copy link
Collaborator

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
Copy link
Collaborator

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>
Copy link
Collaborator

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.

@anjapetkovic anjapetkovic marked this pull request as ready for review April 11, 2025 13:02
@anjapetkovic
Copy link
Author

The readme and the docs are now updated according to Zoe's suggestions.

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