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

Remove default.nix and link to external package #231

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

vdemeester
Copy link
Contributor

Signed-off-by: Vincent Demeester [email protected]

Copy link
Owner

@chmouel chmouel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we keep the nix gha as well ?

@vdemeester
Copy link
Contributor Author

Ah still there ? We can remove yes.

@vdemeester
Copy link
Contributor Author

@chmouel yeah there is not nix gha anymore in the repository I think.

@chmouel chmouel merged commit d15fd55 into chmouel:main Dec 1, 2023
4 checks passed
@vdemeester vdemeester deleted the remove-nix-files branch December 1, 2023 13:54
@06kellyjac
Copy link

It should probably also mention nixpkgs too :)

@vdemeester
Copy link
Contributor Author

It should probably also mention nixpkgs too :)

What do you mean ? Is it packaged in nixpkgs ?

@vdemeester
Copy link
Contributor Author

(If yes, I can remove it from my repo and rely only on upstream 🤭🤭🤭)

@vdemeester
Copy link
Contributor Author

Oh yeah it is !! 🎉👏🏼 @06kellyjac

@chmouel
Copy link
Owner

chmouel commented Dec 5, 2023

@vdemeester would you like to change this to use nixpkgs then?

(and thanks @06kellyjac 🙏🏻 )

@vdemeester
Copy link
Contributor Author

@vdemeester would you like to change this to use nixpkgs then?

(and thanks @06kellyjac 🙏🏻 )

yes I will.

@vdemeester
Copy link
Contributor Author

@chmouel #233

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