You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by "mbeckerle (via GitHub)" <gi...@apache.org> on 2023/02/24 17:02:56 UTC

[GitHub] [daffodil] mbeckerle commented on a diff in pull request #972: Update occursIndex to skip absent elements

mbeckerle commented on code in PR #972:
URL: https://github.com/apache/daffodil/pull/972#discussion_r1117324737


##########
daffodil-test/src/test/scala/org/apache/daffodil/usertests/TestSepTests.scala:
##########
@@ -70,6 +70,8 @@ class TestSepTests {
   @Test def test_sep_ssp_never_5(): Unit = { runner.runOneTest("test_sep_ssp_never_5") }
 
   // DAFFODIL-2791
-  @Test def test_treatAsMissing_occursIndex(): Unit = { runner.runOneTest("test_treatAsMissing_occursIndex") }

Review Comment:
   This line doesn't seem too long to me. Fits fine on my screen for github code review and my browser is zoomed to 133%.
   
   Should we allow longer lines? 
   



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