[PR] Use scope config for compute-engine auth [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
mdedetrich opened a new pull request, #313: URL: https://github.com/apache/incubator-pekko-connectors/pull/313 Currently when getting Google credentials using the `ComputeEngineCredentials` method, it ignores the [`scopes`](https://github.com/apache/incubator-pekko-connectors/blob/1fe3474d6

[PR] Remove duplicate scopes value in reference.conf [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
mdedetrich opened a new pull request, #314: URL: https://github.com/apache/incubator-pekko-connectors/pull/314 While working on https://github.com/apache/incubator-pekko-connectors/pull/313 I realized that we have duplicate scope configuration settings, i.e. one [here](https://github.com/a

Re: [PR] Remove duplicate scopes value in reference.conf [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #314: URL: https://github.com/apache/incubator-pekko-connectors/pull/314#issuecomment-1890886561 Pinging @armanbilge since he originally wrote this, not sure if you wanted to give your 2c -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] Fix some non-standard comments [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #939: URL: https://github.com/apache/incubator-pekko/pull/939#discussion_r1451686749 ## actor/src/main/scala/org/apache/pekko/actor/FSM.scala: ## @@ -743,29 +743,29 @@ trait FSM[S, D] extends Actor with Listeners with ActorLogging { case x

Re: [PR] Fix some non-standard comments [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #939: URL: https://github.com/apache/incubator-pekko/pull/939#discussion_r1451686749 ## actor/src/main/scala/org/apache/pekko/actor/FSM.scala: ## @@ -743,29 +743,29 @@ trait FSM[S, D] extends Actor with Listeners with ActorLogging { case x

Re: [PR] Remove duplicate scopes value in reference.conf [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
pjfanning commented on PR #314: URL: https://github.com/apache/incubator-pekko-connectors/pull/314#issuecomment-1890890839 you'll need something like https://github.com/apache/incubator-pekko-connectors/pull/311/files#diff-5634c415cd8c8504fdb973a3ed092300b43c4b8fc1e184f7249eb29a55511f91R16

Re: [PR] Use scope config for compute-engine auth [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
pjfanning commented on PR #313: URL: https://github.com/apache/incubator-pekko-connectors/pull/313#issuecomment-1890891264 you'll need something like https://github.com/apache/incubator-pekko-connectors/pull/311/files#diff-5634c415cd8c8504fdb973a3ed092300b43c4b8fc1e184f7249eb29a55511f91R16

Re: [PR] Use scope config for compute-engine auth [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #313: URL: https://github.com/apache/incubator-pekko-connectors/pull/313#discussion_r1451689930 ## google-common/src/main/scala/org/apache/pekko/stream/connectors/google/auth/ComputeEngineCredentials.scala: ## @@ -25,18 +25,19 @@ import scala.conc

Re: [PR] feat: Add mapWithResource stream operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #931: URL: https://github.com/apache/incubator-pekko/pull/931#discussion_r1451690122 ## stream-tests/src/test/scala/org/apache/pekko/stream/scaladsl/FlowMapWithResourceSpec.scala: ## @@ -0,0 +1,416 @@ +/* + * Licensed to the Apache Software Foundat

Re: [PR] Remove duplicate scopes value in reference.conf [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #314: URL: https://github.com/apache/incubator-pekko-connectors/pull/314#issuecomment-1890893507 This PR doesn't have any MiMa impact, ill just rebase when 1.0.2 is released -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Use scope config for compute-engine auth [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
mdedetrich commented on code in PR #313: URL: https://github.com/apache/incubator-pekko-connectors/pull/313#discussion_r1451690544 ## google-common/src/main/scala/org/apache/pekko/stream/connectors/google/auth/ComputeEngineCredentials.scala: ## @@ -25,18 +25,19 @@ import scala.

Re: [PR] Fix some non-standard comments [incubator-pekko]

2024-01-14 Thread via GitHub
laglangyue commented on code in PR #939: URL: https://github.com/apache/incubator-pekko/pull/939#discussion_r1451690749 ## actor/src/main/scala/org/apache/pekko/actor/FSM.scala: ## @@ -743,29 +743,29 @@ trait FSM[S, D] extends Actor with Listeners with ActorLogging { case

Re: [PR] feat: Add mapWithResource stream operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #931: URL: https://github.com/apache/incubator-pekko/pull/931#discussion_r1451694837 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -1082,7 +1082,47 @@ trait FlowOps[+Out, +Mat] { * @param onComplete a function that

Re: [PR] feat: Add flatten operator. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #937: URL: https://github.com/apache/incubator-pekko/pull/937#discussion_r1451702538 ## docs/src/main/paradox/stream/operators/index.md: ## @@ -61,6 +61,7 @@ These built-in sinks are available from @scala[`org.apache.pekko.stream.scaladsl |Sin

Re: [PR] use StandardCharsets [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
laglangyue commented on PR #312: URL: https://github.com/apache/incubator-pekko-connectors/pull/312#issuecomment-1890912410 LGTM -- 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 comm

Re: [PR] feat: Add mapWithResource stream operator. [incubator-pekko]

2024-01-14 Thread via GitHub
laglangyue commented on code in PR #931: URL: https://github.com/apache/incubator-pekko/pull/931#discussion_r1451704337 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -1082,7 +1082,47 @@ trait FlowOps[+Out, +Mat] { * @param onComplete a function

Re: [PR] feat: Add mapWithResource stream operator. [incubator-pekko]

2024-01-14 Thread via GitHub
laglangyue commented on code in PR #931: URL: https://github.com/apache/incubator-pekko/pull/931#discussion_r1451704337 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -1082,7 +1082,47 @@ trait FlowOps[+Out, +Mat] { * @param onComplete a function

[PR] chore: Add foreach to index.md [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #941: URL: https://github.com/apache/incubator-pekko/pull/941 Motivation: Fix the `foreach` missing in index.md -- 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 t

Re: [PR] chore: Add foreach to index.md [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #941: URL: https://github.com/apache/incubator-pekko/pull/941 -- 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: notifications-unsubscr...

Re: [PR] chore: Add foreach to index.md [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on PR #941: URL: https://github.com/apache/incubator-pekko/pull/941#issuecomment-1890920625 thanks @He-Pin - could you create a backport PR for this too? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Remove duplicate scopes value in reference.conf [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #314: URL: https://github.com/apache/incubator-pekko-connectors/pull/314#discussion_r1451710311 ## google-common/src/main/scala/org/apache/pekko/stream/connectors/google/auth/GoogleOAuth2.scala: ## @@ -37,7 +37,7 @@ private[auth] object GoogleOAut

Re: [PR] Remove duplicate scopes value in reference.conf [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
mdedetrich commented on code in PR #314: URL: https://github.com/apache/incubator-pekko-connectors/pull/314#discussion_r1451710479 ## google-common/src/main/scala/org/apache/pekko/stream/connectors/google/auth/GoogleOAuth2.scala: ## @@ -37,7 +37,7 @@ private[auth] object Google

Re: [PR] feat: Add mapWithResource stream operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #931: URL: https://github.com/apache/incubator-pekko/pull/931#discussion_r1451714263 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -1082,7 +1082,47 @@ trait FlowOps[+Out, +Mat] { * @param onComplete a function that

Re: [PR] Fix some non-standard comments [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning merged PR #939: URL: https://github.com/apache/incubator-pekko/pull/939 -- 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: notifications-unsubscr

Re: [PR] Fix some non-standard comments [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #939: URL: https://github.com/apache/incubator-pekko/pull/939#issuecomment-1890930220 Thanks , for new contributor from China:) -- 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

Re: [PR] chore: Add foreach to index.md [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #941: URL: https://github.com/apache/incubator-pekko/pull/941#issuecomment-1890931016 > thanks @He-Pin - could you create a backport PR for this too? I will check for that later. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] feat: Add flatten operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #937: URL: https://github.com/apache/incubator-pekko/pull/937 -- 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: notifications-unsubscr...

Re: [I] Feature request: Add flatten to Flow scaladsl [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin closed issue #936: Feature request: Add flatten to Flow scaladsl URL: https://github.com/apache/incubator-pekko/issues/936 -- 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.

Re: [PR] feat: Add mapWithResource stream operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #931: URL: https://github.com/apache/incubator-pekko/pull/931#discussion_r1451723903 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -1082,7 +1082,47 @@ trait FlowOps[+Out, +Mat] { * @param onComplete a function that

[PR] enable compile inline (must not be backported to 1.0.x) [incubator-pekko-http]

2024-01-14 Thread via GitHub
pjfanning opened a new pull request, #418: URL: https://github.com/apache/incubator-pekko-http/pull/418 sbt-pekko-build 0.3.0 enables Scala compile time inlining Do not backport this to 1.0.x branch. 1.0.x branch will need to stick with sbt-pekko-build 0.2.x until we work out how to

[PR] feat: Add dimap operator to Flow. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #942: URL: https://github.com/apache/incubator-pekko/pull/942 Motivation: Add dimap to Flow refs: https://github.com/apache/incubator-pekko/issues/940 result: Can transform the input and output at the same time. What's still missing

[PR] chore: Ignore two pb files from java formatter. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #943: URL: https://github.com/apache/incubator-pekko/pull/943 Motivation: Ignore these two files, which is very annoying. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] feat: Add dimap operator to Flow. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #942: URL: https://github.com/apache/incubator-pekko/pull/942#discussion_r1451734243 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -181,7 +182,22 @@ final class Flow[-In, +Out, +Mat]( * '''Cancels when''' origin

Re: [PR] feat: Add dimap operator to Flow. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #942: URL: https://github.com/apache/incubator-pekko/pull/942#discussion_r1451734424 ## stream/src/main/scala/org/apache/pekko/stream/scaladsl/Flow.scala: ## @@ -181,7 +182,22 @@ final class Flow[-In, +Out, +Mat]( * '''Cancels when''' original

Re: [PR] Update sbt-pekko-build to 0.2.1 [incubator-pekko-http]

2024-01-14 Thread via GitHub
pjfanning commented on PR #400: URL: https://github.com/apache/incubator-pekko-http/pull/400#issuecomment-1890957218 closing due to #418 -- 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 spec

Re: [PR] Update sbt-pekko-build to 0.2.1 [incubator-pekko-http]

2024-01-14 Thread via GitHub
pjfanning closed pull request #400: Update sbt-pekko-build to 0.2.1 URL: https://github.com/apache/incubator-pekko-http/pull/400 -- 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.

[PR] chore: Add @since 1.1.0 to contramap operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #944: URL: https://github.com/apache/incubator-pekko/pull/944 Motivation: Add the `@since 1.1.0` to `contramap` operator -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[I] release notes for 1.1.0(-M0) [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning opened a new issue, #945: URL: https://github.com/apache/incubator-pekko/issues/945 A list of all the PRs is not that helpful since a lot are doc or build related. Changes backported to 1.0.1 and 1.0.2 should not be included either. We needs curated list of changes. Ideally.

Re: [PR] chore: Add @since 1.1.0 to contramap operator. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #944: URL: https://github.com/apache/incubator-pekko/pull/944 -- 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: notifications-unsubscr...

Re: [PR] chore: Ignore two pb files from java formatter. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #943: URL: https://github.com/apache/incubator-pekko/pull/943 -- 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: notifications-unsubscr...

[PR] slf4j2Version 2.0.11 [incubator-pekko-management]

2024-01-14 Thread via GitHub
pjfanning opened a new pull request, #148: URL: https://github.com/apache/incubator-pekko-management/pull/148 (no comment) -- 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. T

Re: [PR] feat: Add dimap operator to Flow. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #942: URL: https://github.com/apache/incubator-pekko/pull/942#issuecomment-1890988386 Will update after local validated, very slow... -- 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

[PR] chore: Fix scaladoc link in contramap.md [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #946: URL: https://github.com/apache/incubator-pekko/pull/946 Motivation: Which needs url-encoded. Result: Correctly rendered -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[PR] feat: Add `create` method to PFBuilder. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #947: URL: https://github.com/apache/incubator-pekko/pull/947 Motivation: `new PFBuilder<>()` is not that fluent. Result: Add a `create` method on `PFBuilder`, alias with others. -- This is an automated message from the Apache Git Service.

Re: [PR] feat: Add `create` method to PFBuilder. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #947: URL: https://github.com/apache/incubator-pekko/pull/947#discussion_r1451769575 ## actor/src/main/java/org/apache/pekko/japi/pf/PFBuilder.java: ## @@ -21,6 +21,11 @@ */ public final class PFBuilder extends AbstractPFBuilder { + /** Creat

[PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #949: URL: https://github.com/apache/incubator-pekko/pull/949 Motivation: Depcate the `JAPI`.java -- 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

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1451775871 ## actor/src/main/scala/org/apache/pekko/japi/JavaAPI.scala: ## @@ -255,13 +254,16 @@ object Util { * Returns an immutable.Seq representing the provided arr

[PR] Fix some non-standard comments (#939) [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning opened a new pull request, #950: URL: https://github.com/apache/incubator-pekko/pull/950 cherry pick 6872f9c8b78ae2fababb97cb4600d67d77b4bbd2 #939 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin closed pull request #949: chore: Deprecate JAPI. URL: https://github.com/apache/incubator-pekko/pull/949 -- 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,

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1451783596 ## actor/src/main/java/org/apache/pekko/japi/JAPI.java: ## @@ -15,8 +15,11 @@ import scala.collection.Seq; +@Deprecated Review Comment: could you you u

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1451783972 ## actor/src/main/java/org/apache/pekko/japi/JAPI.java: ## @@ -15,8 +15,11 @@ import scala.collection.Seq; +@Deprecated Review Comment: Since will require

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1451786219 ## actor/src/main/java/org/apache/pekko/japi/JAPI.java: ## @@ -15,8 +15,14 @@ import scala.collection.Seq; +@Deprecated public class JAPI { + /** + *

Re: [PR] Fix some non-standard comments (#939) [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #950: URL: https://github.com/apache/incubator-pekko/pull/950 -- 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: notifications-unsubscr...

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1451786386 ## actor/src/main/java/org/apache/pekko/japi/JAPI.java: ## @@ -15,8 +15,14 @@ import scala.collection.Seq; +@Deprecated public class JAPI { + /** + *

Re: [PR] fix managerName to use `pekko` [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #587: URL: https://github.com/apache/incubator-pekko/pull/587#discussion_r1451789644 ## remote/src/main/scala/org/apache/pekko/remote/transport/PekkoProtocolTransport.scala: ## @@ -136,7 +136,7 @@ private[remote] class PekkoProtocolTransport(

Re: [PR] slf4j2Version 2.0.11 [incubator-pekko-management]

2024-01-14 Thread via GitHub
pjfanning merged PR #148: URL: https://github.com/apache/incubator-pekko-management/pull/148 -- 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: notificatio

Re: [PR] use StandardCharsets [incubator-pekko-connectors]

2024-01-14 Thread via GitHub
pjfanning merged PR #312: URL: https://github.com/apache/incubator-pekko-connectors/pull/312 -- 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: notificatio

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1451796244 ## actor/src/main/java/org/apache/pekko/japi/JAPI.java: ## @@ -15,8 +15,14 @@ import scala.collection.Seq; +@Deprecated public class JAPI { + /** + *

Re: [PR] enable compile inline (upgrade sbt-pekko-build) [incubator-pekko-http]

2024-01-14 Thread via GitHub
mdedetrich commented on code in PR #418: URL: https://github.com/apache/incubator-pekko-http/pull/418#discussion_r1451805109 ## build.sbt: ## @@ -28,6 +28,9 @@ commands := commands.value.filterNot { command => } } +// pekkoInlineEnabled must be set to false when this is b

Re: [PR] chore: Ignore two pb files from java formatter. [incubator-pekko]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #943: URL: https://github.com/apache/incubator-pekko/pull/943#issuecomment-1891076182 @He-Pin Can you backport this to `1.0.x`? -- 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

[PR] use more of StandardCharsets [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning opened a new pull request, #952: URL: https://github.com/apache/incubator-pekko/pull/952 (no comment) -- 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 unsubscri

Re: [PR] enable compile inline (upgrade sbt-pekko-build) [incubator-pekko-http]

2024-01-14 Thread via GitHub
pjfanning commented on code in PR #418: URL: https://github.com/apache/incubator-pekko-http/pull/418#discussion_r1451808912 ## build.sbt: ## @@ -28,6 +28,9 @@ commands := commands.value.filterNot { command => } } +// pekkoInlineEnabled must be set to false when this is ba

Re: [PR] use more of StandardCharsets [incubator-pekko]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #952: URL: https://github.com/apache/incubator-pekko/pull/952#issuecomment-1891082082 Missing format -- 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 commen

Re: [PR] enable compile inline (upgrade sbt-pekko-build) [incubator-pekko-http]

2024-01-14 Thread via GitHub
mdedetrich commented on code in PR #418: URL: https://github.com/apache/incubator-pekko-http/pull/418#discussion_r1451809325 ## build.sbt: ## @@ -28,6 +28,9 @@ commands := commands.value.filterNot { command => } } +// pekkoInlineEnabled must be set to false when this is b

Re: [PR] enable compile inline (upgrade sbt-pekko-build) [incubator-pekko-http]

2024-01-14 Thread via GitHub
pjfanning merged PR #418: URL: https://github.com/apache/incubator-pekko-http/pull/418 -- 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: notifications-uns

[I] Set `currentVersion` in `PekkoCoreDependency` to `1.1.0-M1` once M1 is released [incubator-pekko-http]

2024-01-14 Thread via GitHub
mdedetrich opened a new issue, #419: URL: https://github.com/apache/incubator-pekko-http/issues/419 As stated in https://github.com/apache/incubator-pekko/pull/857 specifically > It may be that for the Pekko module 1.1.x series they will have to build against Pekko core 1.1.x because

Re: [I] Set `currentVersion` in `PekkoCoreDependency` to `1.1.0-M1` once M1 is released [incubator-pekko-http]

2024-01-14 Thread via GitHub
pjfanning commented on issue #419: URL: https://github.com/apache/incubator-pekko-http/issues/419#issuecomment-1891094928 If we are going to build Pekko HTTP 1.1.x against Pekko 1.1.x then we need to look at creating integration tests that use Pekko HTTP jars but that test with Pekko 1.0.x

Re: [PR] use more of StandardCharsets [incubator-pekko]

2024-01-14 Thread via GitHub
pjfanning merged PR #952: URL: https://github.com/apache/incubator-pekko/pull/952 -- 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: notifications-unsubscr

Re: [I] Set `currentVersion` in `PekkoCoreDependency` to `1.1.0-M1` once M1 is released [incubator-pekko-http]

2024-01-14 Thread via GitHub
mdedetrich commented on issue #419: URL: https://github.com/apache/incubator-pekko-http/issues/419#issuecomment-1891105245 > If we are going to build Pekko HTTP 1.1.x against Pekko 1.1.x then we need to look at creating integration tests that use Pekko HTTP jars but that test with Pekko 1.

Re: [PR] [DRAFT] try sbt-pekko-build [incubator-pekko]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #826: URL: https://github.com/apache/incubator-pekko/pull/826#issuecomment-1891121270 @pjfanning I just pushed 2 commits on this branch. The first one just updates from latest `main`, the second one uses the latest sbt-pekko-build and sets the critical sbt `Settin

Re: [I] Set `currentVersion` in `PekkoCoreDependency` to `1.1.0-M1` once M1 is released [incubator-pekko-http]

2024-01-14 Thread via GitHub
mdedetrich commented on issue #419: URL: https://github.com/apache/incubator-pekko-http/issues/419#issuecomment-1891127429 I asked in https://discord.com/channels/63215047902164/922600050989875282/1196246949129629747 if there is a principled way to do this, otherwise I will add this ca

Re: [PR] chore: Fix scaladoc link in contramap.md [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #946: URL: https://github.com/apache/incubator-pekko/pull/946 -- 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: notifications-unsubscr...

Re: [PR] feat: Add `create` method to PFBuilder. [incubator-pekko]

2024-01-14 Thread via GitHub
laglangyue commented on PR #947: URL: https://github.com/apache/incubator-pekko/pull/947#issuecomment-1891203106 lgtm -- 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 uns

[PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #953: URL: https://github.com/apache/incubator-pekko/pull/953 Motication: Fix compiling error. Result: It compiles now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] Remove repeated word [incubator-pekko]

2024-01-14 Thread via GitHub
aleng1 opened a new pull request, #954: URL: https://github.com/apache/incubator-pekko/pull/954 (no comment) -- 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,

Re: [PR] chore: Remove repeated word in README.md [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #954: URL: https://github.com/apache/incubator-pekko/pull/954 -- 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: notifications-unsubscr...

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #953: URL: https://github.com/apache/incubator-pekko/pull/953 -- 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: notifications-unsubscr...

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #953: URL: https://github.com/apache/incubator-pekko/pull/953#issuecomment-1891238325 Wait, why wasn't this compiling in the first place? The CI should pickup these things and since the source file is in `java-jdk-9` only this could have been caused by having wron

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #953: URL: https://github.com/apache/incubator-pekko/pull/953#issuecomment-1891250009 @mdedetrich I'm re triggering the nightly build, I think that maybe cause of some dependencies change? -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #953: URL: https://github.com/apache/incubator-pekko/pull/953#issuecomment-1891271065 @pjfanning Do you have any idea on this? -- 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

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #953: URL: https://github.com/apache/incubator-pekko/pull/953#issuecomment-1891303240 Triggered at https://github.com/apache/incubator-pekko/actions/runs/7524512274 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] feat: Add dimap operator to Flow. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #942: URL: https://github.com/apache/incubator-pekko/pull/942#issuecomment-1891377925 @mdedetrich @nvollmar You you like to give some input, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on PR #953: URL: https://github.com/apache/incubator-pekko/pull/953#issuecomment-1891378401 @mdedetrich @pjfanning it's works now. -- 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

Re: [PR] feat: Add dimap operator to Flow. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #942: URL: https://github.com/apache/incubator-pekko/pull/942 -- 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: notifications-unsubscr...

[PR] chore: Ignore two pb files from java formatter. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin opened a new pull request, #955: URL: https://github.com/apache/incubator-pekko/pull/955 (cherry picked from commit 485639ef821b5036b521b65d973d698e373721da) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] chore: Ignore two pb files from java formatter. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin merged PR #955: URL: https://github.com/apache/incubator-pekko/pull/955 -- 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: notifications-unsubscr...

Re: [PR] chore: Fix compiling error in AsSubscriber.java and FromPublisher.java [incubator-pekko]

2024-01-14 Thread via GitHub
mdedetrich commented on PR #953: URL: https://github.com/apache/incubator-pekko/pull/953#issuecomment-1891481867 > @mdedetrich @pjfanning it's works now. I understand that it works now, im questioning why it stopped working in the first place. Our CI should be picking up compilation i

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
nvollmar commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1452017338 ## docs/src/test/java-jdk9-only/jdocs/stream/operators/source/AsSubscriber.java: ## @@ -42,13 +39,13 @@ static org.apache.pekko.stream.javadsl.Source> asSubscri

Re: [PR] chore: Deprecate JAPI. [incubator-pekko]

2024-01-14 Thread via GitHub
He-Pin commented on code in PR #949: URL: https://github.com/apache/incubator-pekko/pull/949#discussion_r1452020266 ## docs/src/test/java-jdk9-only/jdocs/stream/operators/source/AsSubscriber.java: ## @@ -42,13 +39,13 @@ static org.apache.pekko.stream.javadsl.Source> asSubscribe