GOODBOY008 opened a new pull request, #768:
URL: https://github.com/apache/fesod/pull/768

   ## Summary
   
   This PR fixes compatibility issues between SLF4J and Logback Classic in the 
test configuration.
   
   ## Changes
   
   ### 1. Added `logback-classic` dependency to `fesod-sheet` module
   - Added `logback-classic` as a test-scoped dependency to provide proper 
SLF4J binding
   - This resolves the "Failed to load class org.slf4j.impl.StaticLoggerBinder" 
warning
   
   ### 2. Cleaned up `logback.xml` configuration
   - Removed invalid Spring-specific configuration (`<springProperty>` tag) 
that is not supported in standard Logback
   - The `springProperty` element requires `logback-spring.xml` and Spring Boot 
environment, which is not available in this test context
   - Simplified to use standard Logback `<property>` element instead
   
   ## Why these changes are needed
   
   - **SLF4J Compatibility**: SLF4J 1.7.x requires a concrete logging 
implementation (binding) at runtime. Without `logback-classic`, only the API is 
available, causing runtime warnings.
   - **Logback Configuration**: The `<springProperty>` element is a Spring Boot 
extension and causes parsing errors when used outside Spring Boot context.
   
   ## Testing
   
   These changes ensure that logging works correctly during test execution 
without any warnings or configuration errors.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to