Skip to content

readme overhaul#1731

Merged
SFJohnson24 merged 14 commits into
mainfrom
readme
May 28, 2026
Merged

readme overhaul#1731
SFJohnson24 merged 14 commits into
mainfrom
readme

Conversation

@SFJohnson24

@SFJohnson24 SFJohnson24 commented May 11, 2026

Copy link
Copy Markdown
Collaborator

readme overhaul with the intention to host the user guide as a streamlined github pages similar to https://cdisc-org.github.io/cdisc-open-rules/#/

@SFJohnson24 SFJohnson24 self-assigned this May 11, 2026
@SFJohnson24 SFJohnson24 linked an issue May 11, 2026 that may be closed by this pull request
Comment thread PYPI.md Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The file is removed from here and moved under docs folder but pyproject.toml still points to this deleted file on line number 9
readme = "PYPI.md"

Comment thread docs/quick-start.md
Comment thread docs/cli-reference.md Outdated
| `-d, --data TEXT` | Path to directory containing dataset files. Only the last value is used if specified multiple times. |
| `-dp, --dataset-path TEXT` | Absolute path to a single dataset file. Can be specified multiple times. |
| `-dxp, --define-xml-path TEXT` | Path to Define-XML. Also via `DEFINE_XML` env var. |
| `-ft, --filetype TEXT` | File extension filter applied to the `-d` directory (e.g. `xpt`). Takes priority over `--dataset-path` when both are provided. |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think the -ft flag does not override the --dataset-path. They are exclusive.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

-ft, --filetype TEXT File extension to filter datasets. Has higher priority than --dataset-path parameter. This is the current text of the readme. I was just copying it

Comment thread docs/cli-reference.md Outdated
Comment thread docs/cli-reference.md Outdated
Comment thread docs/faq.md Outdated
| `SUCCESS` | Rule ran; no issues found. |
| `SKIPPED` | Rule did not run (column/domain not found, schema validation off, outside scope). |
| `ISSUE_REPORTED` | Rule ran; issues were found. |
| `EXECUTION_ERROR` | Rule failed unexpectedly. Check the Issue Details tab for details. |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These are without _ now. It should be EXECUTION ERROR and ISSUE REPORTED. Please update these for consistency throughout the docs.

Comment thread docs/development.md Outdated

@RamilCDISC RamilCDISC left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The PR updates the readme for the repository to be more organized docs. The updated docs are consistent with the previous read me. The validation was done by reviewing and comparing with previous documentation. In addition all the flags, options links were cross validated against the real code and original pages. All formatting was validated by rendering. The pages and side menu was validated by manual validation.

@SFJohnson24 SFJohnson24 merged commit ecb58e7 into main May 28, 2026
14 checks passed
@SFJohnson24 SFJohnson24 deleted the readme branch May 28, 2026 14:48
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.

readme overhaul

3 participants