btashton commented on a change in pull request #1771:
URL: https://github.com/apache/incubator-nuttx/pull/1771#discussion_r487555201
##########
File path: .github/workflows/build.yml
##########
@@ -161,7 +161,7 @@ jobs:
export CCACHE_DIR=`pwd`/ccache
mkdir $CCACHE_DIR
cd sources/testing
- ./cibuild.sh -c -x testlist/${{matrix.boards}}.dat
+ ./cibuild.sh -c testlist/${{matrix.boards}}.dat
Review comment:
Yeah three things that will help with this:
1) Moving to GitHub Container Registry from packages (that ticket I shared).
This will mean we get rid of some of the extra logic in the workflow.
2) GitHub has a planned feature to allow sharing workflows between repos.
3) We can move more of the shared logic into a single GitHub action. This
is going to be worth it longer term, just requires motivation to build the
action in NodeJS which I have not been excited about. Maybe I can set a goal
for myself to have that in the release after the current one.
----------------------------------------------------------------
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:
[email protected]
