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

Adds cachix to CI and flake (like hyberbole) #13

Merged
merged 10 commits into from
Jan 30, 2025
Merged

Conversation

Skyfold
Copy link
Contributor

@Skyfold Skyfold commented Jan 13, 2025

There is no rush on this PR, you can look at this after your vacation. This mirrors the changes in hyperbole:

Updates

  • cabal.project includes examples, so just like hyperbole there is only one project with two cabal files
  • nix flake check also checks nix develop can run cabal --dry-run repl
  • GitHub CI uploads the build dependencies to cachix
  • adds cachix to flake so users can benefit from CI builds
  • runs CI on all branches and all pull requests
    • Only pushes will update the cache since pull requests do not get access to secrets

Requires

  • need to add CACHIX_AUTH_TOKEN as a repository secret
    • It can be found in: settings -> secrets and variables -> Actions -> New repository secret
    • you need to create a cachix account before I can add you (they don't have an invite system)
    • To create the auth token: Settings -> Auth Tokens -> Generate
      • Make sure it is (Write + Read), and (Never) expires

You can merge this without creating a cachix account and setting CACHIX_AUTH_TOKEN since my fork already populated the cache.

@Skyfold
Copy link
Contributor Author

Skyfold commented Jan 27, 2025

The macOS build is failing. I'll let you know when it is fixed so this can be merged.

@Skyfold
Copy link
Contributor Author

Skyfold commented Jan 29, 2025

This is done. Once this is merged I'll put up the hyperbole PR using this as the web-view input.

@Skyfold
Copy link
Contributor Author

Skyfold commented Jan 29, 2025

Notes:

  1. Adds macOS support to the workflow
  2. creates a separate overriddenHaskellPackages to workaround this isssue
  3. Removes warp references in binary as explained here
  4. updates README.md to better explain nix usage.

@seanhess seanhess merged commit ae82448 into seanhess:main Jan 30, 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