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

nix: add lwt #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Alizter
Copy link

@Alizter Alizter commented Oct 11, 2023

We don't need lwt when building notty, but it is required when building the tests. The tests are useful to check how things look.

We don't need lwt when building notty, but it is required when building
the tests. The tests are useful to check how things look.

Signed-off-by: Ali Caglayan <[email protected]>
@Alizter
Copy link
Author

Alizter commented Oct 11, 2023

@rgrinberg It seems GitHub actions from outside this organization are disallowed:
https://github.com/ocaml-dune/notty/actions/runs/6485547263

actions/checkout@v2 and cachix/install-nix-action@v18 are not allowed to be used in 
ocaml-dune/notty. Actions in this workflow must be: within a repository owned by ocaml-dune.

Seems there must be a setting somewhere for this repository about which GitHub actions are allowed to be run.

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.

1 participant