Skip to content

Conversation

@pmachapman
Copy link
Collaborator

@pmachapman pmachapman commented Feb 9, 2026

Second part of fix for sillsdev/machine#372. Requires sillsdev/machine#381 (the CI will fail until that PR is merged).

Example warnings:

warnings: [
  "Invalid chapter number error in project TEA at “MAN 1.” (parallel corpus 69893d8434467c056f3542e5, monolingual corpus 69893d8334467c056f3542e1)",
  "Invalid verse number error in project TEA at “LAO 1:v1” (parallel corpus 69893d8434467c056f3542e5, monolingual corpus 69893d8334467c056f3542e1)"
]

This change is Reviewable

Copy link
Collaborator

@Enkidu93 Enkidu93 left a comment

Choose a reason for hiding this comment

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

If we can add an expected ref on the machine side, that'll need to be added here as well; otherwise,
:lgtm:

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

Copy link
Contributor

@ddaspit ddaspit left a comment

Choose a reason for hiding this comment

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

:lgtm:

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

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