Skip to content

Conversation

@SFJohnson24
Copy link
Collaborator

@SFJohnson24 SFJohnson24 commented Nov 11, 2025

cg0288 PR broke all DDF Codelist term rules-- the issue is due to ct_version getting grabbed and not version. I have ported get_codelist_attributes to using version instead of ct_version and restored the old attribute grab.

this PR restores functionality while continuing to have cg0288 work.
cg0288 report: CORE-Report-2025-11-11T12-17-08.xlsx
ddf00142 - CORE-Report-2025-11-11T12-19-09.xlsx

to test: run ddf rules with codelist_term / codelist_extensible operators
run cg0288 with the below updated logic:
Rule_underscores.json

@SFJohnson24 SFJohnson24 self-assigned this Nov 11, 2025
@SFJohnson24 SFJohnson24 marked this pull request as ready for review November 11, 2025 17:25
Copy link
Collaborator

@ASL-rmarshall ASL-rmarshall left a comment

Choose a reason for hiding this comment

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

I've reviewed the code changes and they look good. I've also run all available USDM CT rules before and after the fix and confirmed that the fix resolves the issue.

@SFJohnson24 SFJohnson24 merged commit d985475 into main Nov 11, 2025
11 checks passed
@SFJohnson24 SFJohnson24 deleted the USDM branch November 11, 2025 18:32
alexfurmenkov pushed a commit that referenced this pull request Nov 13, 2025
* version

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