update published rules action only focuses on filtered standards - #85
Conversation
| exit 0 | ||
| fi | ||
|
|
||
| CORE_IDS_TO_RUN="$(venv/bin/python .github/scripts/filter_core_ids_by_standard.py \ |
There was a problem hiding this comment.
Please let me know if I am misunderstanding the script flow, we can pass the core ids first time the filter_core_ids_by_standard.py script is run. Could you please mention the intention behind running the script twice?
There was a problem hiding this comment.
@RamilCDISC I removed the double call if you could please re-review
RamilCDISC
left a comment
RamilCDISC
left a comment
There was a problem hiding this comment.
The PR updates the action for updating published rules to focus only on filtered datasets. The filtered standards are USDM and TIG. The action has been reviewed for proper implementation and update according to AC. The python script is validated for proper input take and processing of filtered standards. Efficient use of the python script was ensured. The description has a link to sample run.
3 participants
Refer to this action run: https://github.com/cdisc-org/cdisc-open-rules/actions/runs/29511859199