Skip to content

feat: allow host-aspects to work with other user classes#470

Merged
vic merged 5 commits intomainfrom
push-qrkkuusyumtm
Apr 19, 2026
Merged

feat: allow host-aspects to work with other user classes#470
vic merged 5 commits intomainfrom
push-qrkkuusyumtm

Conversation

@vic
Copy link
Copy Markdown
Member

@vic vic commented Apr 19, 2026

Iterates over user.classes instead of just being fixed to homeManager only. Add tests for hjem class.

@vic vic requested a review from sini April 19, 2026 02:59
sini
sini previously approved these changes Apr 19, 2026
Copy link
Copy Markdown
Collaborator

@sini sini left a comment

Choose a reason for hiding this comment

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

LGTM

@vic
Copy link
Copy Markdown
Member Author

vic commented Apr 19, 2026

I'll merge, I'm also using this to test our GH actions now running just ci fast on new PR. and leave slower ci-deep for when allow-ci is present. So that people can iterate faster on their PRs.

@vic vic force-pushed the push-qrkkuusyumtm branch from bfbb705 to bedd518 Compare April 19, 2026 04:27
@vic vic added allow-ci allow all CI integration tests and removed allow-ci allow all CI integration tests labels Apr 19, 2026
@vic vic merged commit 483f5ec into main Apr 19, 2026
10 checks passed
@vic vic deleted the push-qrkkuusyumtm branch April 19, 2026 04:58
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