Skip to content

Conversation

@jtmaxwell3
Copy link
Contributor

@jtmaxwell3 jtmaxwell3 commented Jan 8, 2026

This is in support of https://jira.sil.org/browse/LT-22303. It adds ParagraphParser.EndsWithEOS. It includes unit tests that are relevant for LT-22303.


This change is Reviewable

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

LCM Tests

    16 files  ±0      16 suites  ±0   3m 0s ⏱️ -14s
 2 851 tests +1   2 831 ✅ +1   20 💤 ±0  0 ❌ ±0 
11 352 runs  +4  11 184 ✅ +4  168 💤 ±0  0 ❌ ±0 

Results for commit 26705f6. ± Comparison against base commit 66263ab.

♻️ This comment has been updated with latest results.

@jasonleenaylor
Copy link
Contributor

src/SIL.LCModel/DomainServices/ITextUtils.cs line 278 at r1 (raw file):

		}

		public static bool EndsWithEOS(ITsString text, LcmCache cache)

Please add a method comment explaining the purpose and intended use

Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@jasonleenaylor reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @jtmaxwell3).

@jtmaxwell3 jtmaxwell3 merged commit 9263ed9 into master Jan 9, 2026
5 checks passed
@jtmaxwell3 jtmaxwell3 deleted the LT-22303 branch January 9, 2026 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants