Skip to content

Conversation

@Mic92
Copy link
Member

@Mic92 Mic92 commented Nov 21, 2024

No description provided.

@Mic92 Mic92 marked this pull request as draft November 21, 2024 17:58
@Mic92 Mic92 force-pushed the verity branch 2 times, most recently from ebe8ae3 to cca4d92 Compare November 21, 2024 18:02
@Mic92 Mic92 force-pushed the verity branch 4 times, most recently from 57d9188 to b4dd32e Compare November 29, 2024 13:04
@Mic92 Mic92 force-pushed the verity branch 3 times, most recently from ec8c02b to e601155 Compare November 29, 2024 13:31
@Mic92 Mic92 marked this pull request as ready for review November 29, 2024 13:47
@Mic92 Mic92 force-pushed the verity branch 2 times, most recently from 919ccea to aa00625 Compare November 29, 2024 13:47
@Mic92 Mic92 changed the title [wip] Add declarative unmount Add declarative unmount Nov 29, 2024
@mergify mergify bot mentioned this pull request Nov 29, 2024
Copy link

@pogobanane pogobanane left a comment

Choose a reason for hiding this comment

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

from looking at it, looks good

@Mic92 Mic92 requested a review from a team November 29, 2024 14:10
Copy link
Collaborator

@Lassulus Lassulus left a comment

Choose a reason for hiding this comment

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

changes lgtm, since this just adds new functionality and doesn't touch old one. I would be fine with testing this one "in production" :)

@Lassulus
Copy link
Collaborator

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Nov 29, 2024

queue

🛑 The pull request has been merged manually

The pull request has been merged manually at a75ba3b

@Mic92 Mic92 merged commit a75ba3b into master Nov 29, 2024
3 checks passed
@Mic92 Mic92 deleted the verity branch November 29, 2024 15:39
@Mic92
Copy link
Member Author

Mic92 commented Nov 29, 2024

I actually want to make it a dependency in nixos-anywhere and disko-install

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