[GitHub] [orc] pgaref commented on a change in pull request #593: ORC-708: FIX Snapshot publishing

[GitBox <gi...@apache.org>]

pgaref commented on a change in pull request #593:
URL: https://github.com/apache/orc/pull/593#discussion_r549594677



##########
File path: .github/workflows/publish_snapshot.yml
##########
@@ -17,7 +17,7 @@ jobs:
         java-version: 8
 
     - name: Release Maven package
-      uses: samuelmeuli/action-maven-publish@v1
+      uses: samuelmeuli/action-maven-publish@v1.4.0

Review comment:
       > Are you sure that your claim snapshot publishing is broken by ORC-704?
   You are right, 704 is not related -- just happened that all commits after that failed to publish a snapshot.
   
   Regarding the error message: the confusion comes from the fact that the repo is actually in the apache/* form (thus assumed it was a string/expression evaluation issue). 
   
   >  Do you have any reference why this fix the current error?
   
   No ref here, just wanted to check if this helps with the policy issue above (from the message I assume we shouldn't be seeing the issue and it could be a bug).
   Please let me know If there are other things we could try to fix this, you definitely have more experience GH actions :)  
   




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org