GaoSSR commented on PR #11611:
URL: https://github.com/apache/maven/pull/11611#issuecomment-3701392780
Thanks for the feedback! You're absolutely right about API compatibility.
I'll update the PR to use the deprecate-and-replace approach:
```java
/**
* @deprecated Use {@link #textOccurrencesInLog(List, String)} instead
*/
@Deprecated
public static long textOccurencesInLog(List<String> lines, String text) {
return textOccurrencesInLog(lines, text);
}
public static long textOccurrencesInLog(List<String> lines, String text) {
return lines.stream().filter(line ->
stripAnsi(line).contains(text)).count();
}
```
This way:
- Existing code continues to work (backward compatible)
- IDE will warn users to migrate to the new method
- The old method can be safely removed in a future major version
I'll push the updated changes shortly.
--
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]
