reta commented on code in PR #2237:
URL: https://github.com/apache/cxf/pull/2237#discussion_r1937279650
##
rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/WrapperClassNamingConvention.java:
##
@@ -0,0 +1,148 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) u
reta commented on code in PR #2237:
URL: https://github.com/apache/cxf/pull/2237#discussion_r1937280194
##
rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/WrapperClassNamingConvention.java:
##
@@ -0,0 +1,148 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) u
reta commented on code in PR #2237:
URL: https://github.com/apache/cxf/pull/2237#discussion_r1937284604
##
rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/spi/WrapperClassLoaderTest.java:
##
@@ -115,8 +119,31 @@ public void
testWrapperClassLoaderWhenNoWrappedOperations() t
reta commented on PR #2237:
URL: https://github.com/apache/cxf/pull/2237#issuecomment-2627294305
Love it @ppalaga , thank you, a few nits but LGTM otherwise, @ffang wdyt?
Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
ffang commented on PR #2237:
URL: https://github.com/apache/cxf/pull/2237#issuecomment-2627314942
> Love it @ppalaga , thank you, a few nits but LGTM otherwise, @ffang wdyt?
Thanks!
LGTM, thanks @ppalaga , @reta !
--
This is an automated message from the Apache Git Service.
To resp
ppalaga commented on code in PR #2237:
URL: https://github.com/apache/cxf/pull/2237#discussion_r1937115315
##
rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/WrapperClassUtils.java:
##
@@ -0,0 +1,60 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
ppalaga commented on PR #2237:
URL: https://github.com/apache/cxf/pull/2237#issuecomment-2627037528
6582f43:
* Improved the JavaDoc as suggested by @reta
* Introduced a new extension, called `WrapperClassNamingConvention` for the
naming.
* Provided two implementations of the a
dependabot[bot] opened a new pull request, #2244:
URL: https://github.com/apache/cxf/pull/2244
Bumps [net.sourceforge.pmd:pmd-core](https://github.com/pmd/pmd) from 6.55.0
to 7.10.0.
Release notes
Sourced from https://github.com/pmd/pmd/releases";>net.sourceforge.pmd:pmd-core's
re
reta merged PR #2241:
URL: https://github.com/apache/cxf/pull/2241
--
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: dev-unsubscr...@cxf.apache.org
For q
reta merged PR #2240:
URL: https://github.com/apache/cxf/pull/2240
--
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: dev-unsubscr...@cxf.apache.org
For q
reta merged PR #2242:
URL: https://github.com/apache/cxf/pull/2242
--
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: dev-unsubscr...@cxf.apache.org
For q
dependabot[bot] closed pull request #143: Bump net.sourceforge.pmd:pmd-java
from 6.55.0 to 7.9.0
URL: https://github.com/apache/cxf-build-utils/pull/143
--
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
dependabot[bot] opened a new pull request, #144:
URL: https://github.com/apache/cxf-build-utils/pull/144
Bumps [net.sourceforge.pmd:pmd-java](https://github.com/pmd/pmd) from 6.55.0
to 7.10.0.
Release notes
Sourced from https://github.com/pmd/pmd/releases";>net.sourceforge.pmd:pmd-
dependabot[bot] commented on PR #143:
URL: https://github.com/apache/cxf-build-utils/pull/143#issuecomment-2628507821
Superseded by #144.
--
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 speci
sstremler commented on PR #2245:
URL: https://github.com/apache/cxf/pull/2245#issuecomment-2628556571
> Thank you @sstremler , a few comments but LGTM otherwise!
Thank you, I added the changes you suggested.
--
This is an automated message from the Apache Git Service.
To respond to
reta commented on code in PR #2245:
URL: https://github.com/apache/cxf/pull/2245#discussion_r1938068328
##
rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java:
##
@@ -125,31 +126,36 @@ private String getPrincipal(Message message) {
reta commented on code in PR #2245:
URL: https://github.com/apache/cxf/pull/2245#discussion_r1938070843
##
rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java:
##
@@ -125,31 +126,36 @@ private String getPrincipal(Message message) {
reta commented on PR #2245:
URL: https://github.com/apache/cxf/pull/2245#issuecomment-2628539485
Thank you @sstremler , a few comments but LGTM otherwise!
--
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 t
18 matches
Mail list logo
