[GitHub] [daffodil] tuxji commented on pull request #972: Update occursIndex to skip absent elements

["tuxji (via GitHub)" <gi...@apache.org>]

tuxji commented on PR #972:
URL: https://github.com/apache/daffodil/pull/972#issuecomment-1448439341

   I wonder if it would be worth moving checks that aren't specific to OS and JDK versions into their own "lint" job inside main.yml?  It must be discouraging to make a change to a PR and see 3 out of 6 matrix jobs fail across the board.  The Rat, OSGi, scalafmt, and sonarcloud checks seem like they could run in their own job just fine.  If we run these checks asynchronously in their own job, I suspect we want to allow the matrix jobs to complete even if the lint job fails since we still want to know whether the unit and integration tests pass (the more information the better), yes?


-- 
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: commits-unsubscribe@daffodil.apache.org

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