Alwaysgaurav1 commented on PR #1801:
URL: https://github.com/apache/commons-lang/pull/1801#issuecomment-6086481526

   Hi @garydgregory,
   
   Thanks for running the Copilot review. In response to the feedback:
   
   - **Aligned PR Title & Scope**: Updated the PR title and description to 
reflect the current scope (documentation + regression test only—no new API 
methods added).
   - **Brittle Test Assertions**: Removed the hardcoded epoch millisecond 
assertions in `DateUtilsTest.java`, relying strictly on elapsed hours (`23h` / 
`25h`) and resulting calendar fields across DST transitions.
   - **Clean History**: Branch is cleanly rebased onto the latest master into a 
single linear commit (no merge commits, `changes.xml` untouched).
   - **Verification**: `DateUtilsTest` passes (134/134) with 0 Checkstyle 
violations.
   
   It's ready for whenever you have a chance to take a look. Thanks!


-- 
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]

Reply via email to