Skip to content

feat: Introduce CLI configuration#1177

Merged
gtema merged 1 commit intomainfrom
cli_config
Apr 25, 2025
Merged

feat: Introduce CLI configuration#1177
gtema merged 1 commit intomainfrom
cli_config

Conversation

@gtema
Copy link
Owner

@gtema gtema commented Apr 25, 2025

Prepare for making CLI output configurable using config files (similarly
to the TUI configuration). The config is searched in the
$XDG_CONFIG_DIR/osc/[config.yaml, views.yaml].

This change is only preparing for the next step of passing additional
information from the codegenerator, so it is not feature complete.

Prepare for making CLI output configurable using config files (similarly
to the TUI configuration). The config is searched in the
`$XDG_CONFIG_DIR/osc/[config.yaml, views.yaml]`.

This change is only preparing for the next step of passing additional
information from the codegenerator, so it is not feature complete.
}

/// Get OutputConfig from passed arguments
pub fn from_args_with_resource_key<S: AsRef<str>>(args: &Cli, resource_key: S) -> Self {

Check warning

Code scanning / clippy

associated function from_args_with_resource_key is never used Warning

associated function from_args_with_resource_key is never used
}

/// Get OutputConfig from passed arguments
pub fn from_args_with_resource_key<S: AsRef<str>>(args: &Cli, resource_key: S) -> Self {

Check warning

Code scanning / clippy

associated function from_args_with_resource_key is never used Warning

associated function from_args_with_resource_key is never used
@gtema gtema merged commit e89a2f0 into main Apr 25, 2025
23 checks passed
gtema added a commit that referenced this pull request Apr 25, 2025
Prepare for making CLI output configurable using config files (similarly
to the TUI configuration). The config is searched in the
`$XDG_CONFIG_DIR/osc/[config.yaml, views.yaml]`.

This change is only preparing for the next step of passing additional
information from the codegenerator, so it is not feature complete.
@gtema gtema deleted the cli_config branch April 25, 2025 19:22
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.

1 participant