Re: [PR] Drop RxJava 2 [httpcomponents-core]
rschmitt merged PR #540: URL: https://github.com/apache/httpcomponents-core/pull/540 -- 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]
Re: [PR] Drop RxJava 2 [httpcomponents-core]
rschmitt commented on code in PR #540: URL: https://github.com/apache/httpcomponents-core/pull/540#discussion_r2240224383 ## pom.xml: ## @@ -232,6 +231,7 @@ true true +true Review Comment: No, I changed this by hand. This option appeared to be in the wrong section and wasn't taking effect. Without this option here, japicmp fails because the current version's classpath is missing the RxJava 2 classes (not to be confused with the missing `ReactiveTestUtils` class). Evidently, `ignoreMissingClasses` needs to be set in the build plugin config, not the reporting plugin config. -- 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]
Re: [PR] Drop RxJava 2 [httpcomponents-core]
rschmitt commented on code in PR #540: URL: https://github.com/apache/httpcomponents-core/pull/540#discussion_r2240224383 ## pom.xml: ## @@ -232,6 +231,7 @@ true true +true Review Comment: No, I changed this by hand. This option appeared to be in the wrong section and wasn't taking effect. -- 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]
Re: [PR] Drop RxJava 2 [httpcomponents-core]
ok2c commented on code in PR #540: URL: https://github.com/apache/httpcomponents-core/pull/540#discussion_r2239502351 ## pom.xml: ## @@ -232,6 +231,7 @@ true true +true Review Comment: @rschmitt This is due to formatting, is it not? -- 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]
