With the latest version of OAK (later than ODK 1.5.1) the "Broad match terms with the term 'other' in them." causes a problem (re.error: global flags not at the start of the expression at position 2). @cmungall advised that this is not correct yaml syntax (using the "r".
Overview
This PR:
Updates the lexmatch rules removing the r in front of all match expressions (as per @cmungall advice)
Updates the match clause of the 's rule, which was flagged as illegal yaml by my linter
Pre-merge checklist
Documentation
Was the documentation added/updated under docs/?
[ ] Yes
[X] No, updates to the docs were not necessary after careful consideration
QC
Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after
docker pull obolibrary/odkfull:dev), and no errors occurred?
[ ] Yes
[ ] No, there are no functional (code-related) changes to the pipeline in the PR, so no re-run is necessary
With the latest version of OAK (later than ODK 1.5.1) the "Broad match terms with the term 'other' in them." causes a problem (
re.error: global flags not at the start of the expression at position 2
). @cmungall advised that this is not correct yaml syntax (using the "r".Overview
This PR:
r
in front of all match expressions (as per @cmungall advice)match
clause of the's
rule, which was flagged as illegal yaml by my linterPre-merge checklist
Documentation
Was the documentation added/updated under
docs/
?QC
Was the full pipeline run before submitting this PR using
sh run.sh make build-mondo-ingest
on this branch (afterdocker pull obolibrary/odkfull:dev
), and no errors occurred?New Packages
Were any new Python packages added?
Were any other non-Python packages added?
PR Review and Conversations Resolved
Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?
Additional checklist