ASF GitHub Bot logged work on BEAM-9301:

                Author: ASF GitHub Bot
            Created on: 14/Feb/20 14:35
            Start Date: 14/Feb/20 14:35
    Worklog Time Spent: 10m 
      Work Description: suztomo commented on pull request #10865: [BEAM-9301] 
Checkout the hash of master instead of the branch in beam linkage checker script
URL: https://github.com/apache/beam/pull/10865#discussion_r379462324

 File path: sdks/java/build-tools/beam-linkage-check.sh
 @@ -84,9 +84,9 @@ function runLinkageCheck () {
 BRANCH_COMMIT=`git rev-parse --short=8 HEAD`
 runLinkageCheck $BRANCH_COMMIT
-git checkout master
-git pull
-MASTER_COMMIT=`git rev-parse --short=8 HEAD`
+git fetch
+MASTER_COMMIT=`git rev-parse --short=8 master`
 Review comment:
   MASTER_COMMIT=`git rev-parse --short=8 origin/master`
   local master and remote master branch are different until “git pull”
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:

Issue Time Tracking

    Worklog Id:     (was: 387390)
    Time Spent: 1h 40m  (was: 1.5h)

> Add a script to check Java linkage issues (beam-linkage-check.sh)
> -----------------------------------------------------------------
>                 Key: BEAM-9301
>                 URL: https://issues.apache.org/jira/browse/BEAM-9301
>             Project: Beam
>          Issue Type: Task
>          Components: build-system
>            Reporter: Tomo Suzuki
>            Assignee: Tomo Suzuki
>            Priority: Major
>             Fix For: 2.20.0
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
> https://github.com/apache/beam/pull/10769#issuecomment-584571787
> bq. @suztomo can you contribute this script maybe into Beam's build-tools 
> directory so we can improve it a bit for further use?
> This is a temporary solution before exclusion rules in Linkage Checker 
> (BEAM-9206) are implemented.

This message was sent by Atlassian Jira

Reply via email to