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 Nix Flake #130

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

Add Nix Flake #130

wants to merge 77 commits into from

Conversation

justDeeevin
Copy link

@justDeeevin justDeeevin commented Nov 10, 2024

See #140.
In addition to packaging the app, this flake provides a home-manager module with which users can configure Posting in Nix.

@justDeeevin
Copy link
Author

Draft because home-manager module is WIP. The package itself builds and runs.

@noahfraiture
Copy link

Hey, I'm starting nix. Do you have an example of how you install posting in your nix configuration with this ? Do you simply have to add this repo in inputs of a flake ?

@justDeeevin
Copy link
Author

  1. add the repo to inputs
  2. add specialArgs = {inherit inputs;} as an attribute in the nixosSystem function call
  3. add inputs argument to your config module
  4. add inputs.posting.packages.<system>.default to your systemPackages (where <system> is your system, probably x86_64-linux)

See this reddit thread, or look at my nixos flake for more.

@justDeeevin justDeeevin marked this pull request as ready for review November 13, 2024 00:26
@darrenburns
Copy link
Owner

I've never used Nix and have no clue how to test this, so would appreciate if someone else more familiar with it is able to chime in here.

@mhkarimi1383
Copy link

Awesome
Can you also provide documentation related to how to use it?

@justDeeevin
Copy link
Author

In the README?

@mhkarimi1383
Copy link

Yes, it should be added into tge How to install part

@justDeeevin
Copy link
Author

Apologies for the flood of commits--I'm not aware of any way to test the home manager module apart from using it in my system configuration, which means that I have to have the changes on github to test them... this probably should get squashed before merging to be honest

I think this state of docs is good. Thoughts?

@justDeeevin justDeeevin marked this pull request as ready for review November 20, 2024 16:35
@mhkarimi1383
Copy link

I think a full example would be great

@justDeeevin
Copy link
Author

I've added an example that replicates the example configuration used in the other docs, along with a custom theme also pulled from the documentation.

@mhkarimi1383
Copy link

mhkarimi1383 commented Nov 20, 2024

Awesome

Addimg a sample flake config would be also great :)
It will make it easy to use

@justDeeevin
Copy link
Author

README felt like the right place, but I could repurpose home_manager.md to just be nix.md if you think it belongs outside of the README

@justDeeevin
Copy link
Author

Alright, that should be everything!

@mhkarimi1383
Copy link

Awesome waiting for merge :)

i think i messed up the damn theme enum again
@justDeeevin
Copy link
Author

Since this is going stale, I've made a flake-only repo.

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.

4 participants