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

Support command line arguments as a configuration source #22

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

51-code
Copy link
Contributor

@51-code 51-code commented Nov 18, 2024

Fixes #7 .

Adds support for command line.

Checks that the arguments are in "key=value" format. A copy of the array is taken in the constructor to make the object immutable.

Remove unnecessary code in test

Support command line arguments as a configuration source
@51-code 51-code added the enhancement New feature or request label Nov 18, 2024
@51-code 51-code requested a review from elliVM November 18, 2024 08:40
@51-code 51-code self-assigned this Nov 18, 2024
@51-code 51-code changed the title Support command line arguments as a configuration Support command line arguments as a configuration source Nov 18, 2024
Copy link

@elliVM elliVM left a comment

Choose a reason for hiding this comment

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

LGTM

@51-code 51-code requested a review from kortemik November 18, 2024 12:06
@kortemik kortemik merged commit ac52f38 into teragrep:main Nov 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for command line
3 participants