-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial Release #3
Conversation
Add assertion helper functions and tests - Implement various assertion helper functions such as `IsDecreasingf`, `IsIncreasing`, `IsNonDecreasing`, `IsNonIncreasing`, `IsType`, `JSONEq`, `Len`, `Less`, `Negative`, `Never`, `Nil`, `NoDirExists`, `NoError`, `NoFileExists`, `NotContains`, `NotEmpty`, `NotEqual`, `NotErrorIs`, `NotImplements`, `NotNil`, `NotPanics`, `NotRegexp`, `NotSame`, `NotSubset`, `NotZero`, `Panics`, `Positive`, `Regexp`, `Same`, `Subset`, `True`, `WithinDuration`, `WithinRange`, `YAMLEq`, and `Zero`. - Add corresponding test cases for the new assertion functions. - Ensure all assertion functions call `t.FailNow()` if the assertion fails. - Update `require` package with new assertion functions. - Add `TestGetLevelFromEnv` to test environment variable-based log level retrieval.
Removed outdated `README.md` in `log/slogx` and streamlined `README.md` in the root directory. Added badges for GoDoc, Go Report Card, and License, and included a basic usage example. Updated `CHANGELOG.md` to mark the initial release and updated `CODEOWNERS` with actual usernames.
- Implement an HTTP server that logs requests with context support. - Use `slogx` to build a logger with JSON format and context handler. - Manage log levels using environment variables with `LevelManager`.
suggested changes for example
- Added missing `TestContextLoggerWithMultipleUpdates` test case to context-handler_test.go
- Added context-aware/example.go to demonstrate context-aware logging - Added level-manager/example.go to demonstrate changing log levels at runtime. THis also demonstrates having multiple loggers
…t.go and logger-builder_test.go - Removed example.go - Enhance README.md to include different usages
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.
Welcome to the project! Verify your commit history (git log) to ensure it does NOT contain public email addresses!
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.
See inline comments please.
- CHANGELOG.md - CODE_OF_CONDUCT.md - LICENSE - NOTICE - level-manager.go
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.
Looks good - thanks!
- added examples to README.md - corrected version tag in CHANGELOG.md
- Moved some of the description to a Features section.
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.
see in-line comments
No description provided.