Skip to content

readme overhaul#1731

Open
SFJohnson24 wants to merge 12 commits into
mainfrom
readme
Open

readme overhaul#1731
SFJohnson24 wants to merge 12 commits into
mainfrom
readme

Conversation

@SFJohnson24
Copy link
Copy Markdown
Collaborator

@SFJohnson24 SFJohnson24 commented May 11, 2026

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
Copy link
Copy Markdown
Collaborator

@RamilCDISC RamilCDISC left a comment

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.

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