stevedlawrence commented on code in PR #1602:
URL: https://github.com/apache/daffodil-vscode/pull/1602#discussion_r2760749525
##########
.github/workflows/CI.yml:
##########
@@ -107,6 +110,7 @@ jobs:
build-test-package:
name: "Build, Test, and Package (OS: ${{ matrix.os }}, Node: ${{
matrix.node }}, Java: ${{ matrix.java_version }}, VS Code: ${{ matrix.vscode
}})"
strategy:
+ max-parallel: 15
Review Comment:
Note that the matrix for this job results in 44 jobs. With a max-parallel of
15, this is expected to take approximately 3 times longer. Maybe that's fine if
each job is only a couple of minutes. But it might be worth taking a look at
this matrix to see if all of these jobs are really necessary, or if some could
be removed and still provide confidence that a PR does not break anything.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
