mdedetrich commented on code in PR #333:
URL:
https://github.com/apache/incubator-pekko-http/pull/333#discussion_r1364153053
##########
project/PekkoDependency.scala:
##########
@@ -44,6 +44,7 @@ object PekkoDependency {
case None =>
Option(System.getProperty("pekko.http.build.pekko.version")) match {
case Some("main") => mainSnapshot
+ case Some("snapshot-1.0.x") => snapshot10x
Review Comment:
> `main` is the name of the branch we are using for 1.1.x so calling the
option to use a 1.0.x snapshot something with the word `main` in it would be
very confusing
No its not, `main` points to the latest `main` branch which are snapshots
and the `mainSnapshot` function finds the latest snapshot version from that
`main` branch (thats why its called `main`). There is an argument the naming is
confusing but if we want to change that lets do it in another PR
--
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]
