You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "mdedetrich (via GitHub)" <gi...@apache.org> on 2023/02/02 22:16:38 UTC
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich opened a new pull request, #5: Fix scalafmt check context
mdedetrich opened a new pull request, #5:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/5
Judging from another project I maintain I think this should be correct version?
--
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-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich commented on pull request #5: Fix scalafmt check context
Posted by "mdedetrich (via GitHub)" <gi...@apache.org>.
mdedetrich commented on PR #5:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/5#issuecomment-1414454188
> Are we stuck in a Catch-22? We can't merge this because the status check won't pass. We can't commit to main branch directly - so we can't avoid the PR problem that way. We can't force push to main either.
Yes, I am creating an INFRA ticket to solve 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 go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] pjfanning commented on pull request #5: Fix scalafmt check context
Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #5:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/5#issuecomment-1421582414
I think .asf.yaml only has an effect if you commit the change in main branch.
--
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-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich commented on pull request #5: Fix scalafmt check context
Posted by "mdedetrich (via GitHub)" <gi...@apache.org>.
mdedetrich commented on PR #5:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/5#issuecomment-1421565559
> lgtm - but we may need to get back to INFRA if this change doesn't work
Unless we can confirm that this works, I would prefer to test this on a branch that is not main so we don't freeze main again.
--
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-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] mdedetrich merged pull request #5: Fix scalafmt check context
Posted by "mdedetrich (via GitHub)" <gi...@apache.org>.
mdedetrich merged PR #5:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/5
--
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-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
[GitHub] [incubator-pekko-sbt-paradox] pjfanning commented on pull request #5: Fix scalafmt check context
Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #5:
URL: https://github.com/apache/incubator-pekko-sbt-paradox/pull/5#issuecomment-1414453790
Are we stuck in a Catch-22? We can't merge this because the status check won't pass. We can't commit to main branch directly - so we can't avoid the PR problem that way. We can't force push to main either.
--
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-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org