Conversation
Change-Id: I01a58d2badb9b4d863a91e975499aa49a7d47354
Change-Id: I1ddf2f34733cb72d23b6c705a4dfa653c6b53a12
Change-Id: Ifbc74826caebdae3e0556e5351672f11b38612e5
Change-Id: Ice39b6d3508347f41aface7235940cbd7cdca81c
Change-Id: Id936cb91e838b813b926b5acf18095c7ae295ac7
Change-Id: Ie276229faf998d0ab45c6d3b9f9cfdcd4131fe98
Change-Id: Id7ea53849eab70d91e0dfa43a461c6556aa2856c
Change-Id: Icc7e54b7a50f5d8027c16eaf0efb38186d4965e3
AndyBlack
changed the title
AndyBlack
changed the title
NUnit Tests 1 files ± 0 1 suites ±0 5m 58s ⏱️ -1s For more details on these failures, see this check. Results for commit 85c46b8. ± Comparison against base commit c450a55. ♻️ This comment has been updated with latest results. |
papeh
reviewed
Mar 25, 2026
Contributor
papeh
left a comment
papeh
left a comment
There was a problem hiding this comment.
What I've spot-checked from 5 million lines looks good. LGTM when tests are passing.
Src/Utilities/pcpatrflex/DisambiguateInFLExDB/FLExDBExtractor.cs
Outdated
Show resolved
Hide resolved
Change-Id: Ib739390df9ef2edad7bdee19449f25d32ae098a5
Change-Id: Ifd2278bef9843b4ad7e75e3b961f653a2c407402
Change-Id: I0a5c654c1cb0a7b1c31e47c851cd1b508c6f1d95
Change-Id: I7631e225898ae9471c308cde298b429059b97a12
Change-Id: I34e56f72f95be84433111305ed3c8286d2cabf7f
papeh
reviewed
Mar 27, 2026
Src/Utilities/pcpatrflex/DisambiguateInFLExDB/DisambiguateInFLExDBTests/PcPatrInvokerTests.cs
Outdated
Show resolved
Hide resolved
Src/Utilities/pcpatrflex/ExtractAnaFilesFromTexts/ExtractAnaFilesFromTextsForm.cs
Outdated
Show resolved
Hide resolved
Change-Id: I23246e3d18c99655b12394d08ba813688adb4fe7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is for LT-22418 (Move PCPATR with FLEx into the FLEx repo).
This contains three FLEx add-on utilities that have been in production use by a few FLEx users for some time now. The changes here moved what was in the GitHub repository at https://github.com/sillsdev/pcpatrflex/tree/master to Src\Utilities. The utilities are:
Use PcPatr with FLEx: A tool to disambiguate a FLEx text by using a syntactic grammar with PcPatr.
Extract Ana Files from FLEx Texts: A tool to create a CARLA ANA file for a given text in FLEx.
Use TonePars with FLEx: A tool to use the TonePars tool to diambiguate texts in FLEx.
There also is a ReadMe.txt file pointing to the soon-to-be-archived repository mentioned above in case some one needs to check its history.
Note that the code in that repository that is now in FLEx has never undergone a human code review. Since there is so much code, I tried to put each project in its own commit.
This change is