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

fix example #100

Closed
wants to merge 1 commit into from
Closed

fix example #100

wants to merge 1 commit into from

Conversation

Asqiir
Copy link
Contributor

@Asqiir Asqiir commented Mar 30, 2024

This is a fix for the errors noticed by @dvaerum in #84.

I had to update the stateversion (18.09 would have been default, but it did not compile with the old version but does with the new).

The example now works again.

@Asqiir
Copy link
Contributor Author

Asqiir commented Mar 30, 2024

This should also close #57.

@Asqiir
Copy link
Contributor Author

Asqiir commented Mar 30, 2024

@magnouvean I hope it's ok to ping you (or should I just wait with PRs like that?)

@magnouvean
Copy link
Collaborator

Yeah no problem about it. I have been reworking the documentation and examples a bit in my fork at https://github.com/magnouvean/plasma-manager/tree/documentation-updates, and among the changes have been in flake.nix. There I've rewritten the flake.nix quite a lot. I think it should be ready in a few days and we do one big pr for improved documentation. Therefore I think there's not much point in fixing this right now as it probably will just be changed in a couple days anyway. I'll open a draft pr right away and we can work there directly

@Asqiir
Copy link
Contributor Author

Asqiir commented Mar 31, 2024

Closing this in favour of #104

@Asqiir Asqiir closed this Mar 31, 2024
@Asqiir Asqiir deleted the fix-example branch June 5, 2024 20:27
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