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

Fix linter warnings #10

Merged
merged 1 commit into from
Aug 22, 2024
Merged

Fix linter warnings #10

merged 1 commit into from
Aug 22, 2024

Commits on Aug 22, 2024

  1. chore: Add Out field to related functions for exhaustruct lint warning

    chore: add required fields to related functions for exhaustruct lint warning
    
    chore: remove nolint:varnamelen  comments
    
    because rule disabled in config already
    
    feat(logfx): improve replacerGenerator function
    
    - add key-value attributes in prettyMode
    - change switch statement to if at error handling
    
    feat(logfx): improve error handling in Handler
    
    - Add error wrapping to Handle method
    - Update WithAttrs and WithGroup methods to retain writer and config fields
    
    revert "chore: Add Out field to related functions for exhaustruct lint warning"
    
    This reverts commit 1ef434f.
    
    Revert "chore: add required fields to related functions for exhaustruct lint warning"
    
    This reverts commit 1ad0dfc.
    
    refactor(logfx): improve replacerGenerator function
    
    - Change key-value attributes in prettyMode to empty
    code-a-man authored and eser committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    e45816d View commit details
    Browse the repository at this point in the history