Skip to content

CDM v5.4 documentation migration to quarto and bugfixes - #800

Merged
clairblacketer merged 30 commits into
mainfrom
develop
Aug 4, 2026
Merged

CDM v5.4 documentation migration to quarto and bugfixes#800
clairblacketer merged 30 commits into
mainfrom
develop

Conversation

@clairblacketer

Copy link
Copy Markdown
Contributor

This is a bugfix release of CDM v5.4 prior to releasing CDM v5.5. It also includes the migration of documentation from rMarkdown to Quarto thanks to @MaximMoinat.

@MaximMoinat MaximMoinat 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.

Looks good. Minor comment on further cleaning the field level csvs.

Comment thread inst/csv/OMOP_CDMv5.4_Field_Level.csv

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 propose to remove the v6.0 documentation. It should not be used and having it here creates maintenance overhead and might confuse people.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have kept it because some folks in the community requested it. I did remove links to 6.0 from the website though so the only way to get to it is direct urls. Is that enough do you think?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we will discuss on the 8/5 CDM call and, if the community agrees, we will remove the CDM v6.0 documentation when we release CDM v5.5

@clairblacketer
clairblacketer merged commit 746a15e into main Aug 4, 2026
1 of 7 checks passed
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.

2 participants