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

Add flake for dev environment on nix #65

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

Conversation

fidgetingbits
Copy link
Contributor

This is what I used to get a build environment working on nix, maybe not the best as I'm new to nix, but works.

@fidgetingbits
Copy link
Contributor Author

fwiw I was looking into the next step of adding new language support and it made me remember you are using a monorepo for all of the different cursorless stuff, so it may make more sense to have one flake in the base that allows development for everything, but I'll look into it more.

@fidgetingbits
Copy link
Contributor Author

fidgetingbits commented Sep 29, 2023

Since this tool isn't included in the cursorless monrepo, could still add this flake if you decide you want them. Although as with the cursorless one, I'll need to remove node2nix from this one too. But will do it re-test another day. Also I removed the .envrc, so need to update the README to tweak the instructions still.

@fidgetingbits
Copy link
Contributor Author

Actually now that we don't use the vendored web-tree-sitter, it simplifies stuff on nix, so don't need any of the instructions for development in the readme in the end. I've removed the other unneeded dependencies from the flake as well.

@pokey
Copy link
Member

pokey commented Oct 16, 2023

Fwiw we are now vendoring web-tree-sitter again 😅. See #69

@auscompgeek
Copy link
Member

Well, if we can get everyone working on this on nix, we should resolve everyone's weird "it works on my machine" build issues related to node-gyp

flake.nix Outdated Show resolved Hide resolved
@pokey
Copy link
Member

pokey commented Jan 19, 2024

just wanted to check this one wasn't waiting on a review from me

@auscompgeek
Copy link
Member

Still waiting for steam-run to be removed from the dev shell packages.

@fidgetingbits
Copy link
Contributor Author

Removed it

@pokey
Copy link
Member

pokey commented Jan 20, 2024

great. if @auscompgeek is happy I'm happy

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.

3 participants