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

Add support for configuration from file source and default configs #10

Merged
merged 9 commits into from
Nov 7, 2024

Conversation

51-code
Copy link
Contributor

@51-code 51-code commented Oct 29, 2024

Fixes issues #5 and #9 .

Provides an unmodifiable map from file source and a decorator for default configurations in case the file is not found.

Adds some dependencies for logging and testing.

@51-code 51-code added the enhancement New feature or request label Oct 29, 2024
@51-code 51-code requested a review from eemhu October 29, 2024 07:58
eemhu
eemhu previously approved these changes Oct 29, 2024
Copy link

@eemhu eemhu 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 October 29, 2024 08:14
@51-code 51-code requested a review from kortemik November 1, 2024 09:43
@51-code 51-code requested a review from kortemik November 7, 2024 08:59
@kortemik kortemik merged commit a34962f into teragrep:main Nov 7, 2024
1 check passed
This was linked to issues Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default configurations Add support for path properties
4 participants