nuttx-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on a change in pull request #229: Restore Git Credentials on NuttX repo
Date Tue, 05 May 2020 06:35:15 GMT

xiaoxiang781216 commented on a change in pull request #229:
URL: https://github.com/apache/incubator-nuttx-apps/pull/229#discussion_r419891511



##########
File path: .github/workflows/build.yml
##########
@@ -162,13 +162,23 @@ jobs:
           retry_wait_seconds: 10
           command: docker pull docker.pkg.github.com/apache/incubator-nuttx-testing/nuttx-ci-linux
 
+      - name: Export NuttX Repo SHA
+        run:  echo "::set-env name=nuttx_sha::`git -C sources/nuttx rev-parse HEAD`"
+      - name: Refresh Git Credentials
+        uses: actions/checkout@v2
+        with:
+          repository: apache/incubator-nuttx

Review comment:
       The current method require each job(20~):
   1.Clean the repo and reset all refs
   2.Fetch the tag from server
   Can we save these two step if we move the tag fetch to 'Fetch-Source->Checkout nuttx
repo'? I am wondering which method is more fast and stable.
   




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



Mime
View raw message