-
Notifications
You must be signed in to change notification settings - Fork 45
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
Linux user add #218
Linux user add #218
Conversation
Codecov Report
@@ Coverage Diff @@
## main #218 +/- ##
==========================================
- Coverage 47.00% 46.45% -0.55%
==========================================
Files 68 69 +1
Lines 2136 2161 +25
==========================================
Hits 1004 1004
- Misses 1132 1157 +25
Continue to review full report at Codecov.
|
A comment for the review. I use compiler directives for Linux and non-Linux since keeping it all in a match body to match against os_info Types would be exhaustive and need to be kept up to date as os_info add more distros to it's types. |
@@ -6,6 +6,8 @@ use self::none::NoneUserProvider; | |||
use super::UserVariant; | |||
use schemars::JsonSchema; | |||
use serde::{Deserialize, Serialize}; | |||
mod linux; | |||
use self::linux::LinuxUserProvider; | |||
|
|||
#[derive(JsonSchema, Clone, Debug, Serialize, Deserialize)] | |||
pub enum UserProviders { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A minor nit. You could get rid of ...UserProvider
. I think, that's why clippy is complaining.
But you can merge if you solve it later 👍
I'm submitting a
What is the current behaviour?
Comtrya uses the NoneProvider on linux distibutions.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
What is the expected behavior?
Be able to use the user add action to add users to a linux system
What is the motivation / use case for changing the behavior?
Feature parity with FreeBSD on Comtrya and useful feature for all supported operating systems for Comtrya
Please tell us about your environment:
OpenSUSE TumbleWeed
Version (
comtrya --version
): comtrya 0.7.4Operating system:
OpenSUSE Tumbleweed