Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature Request: Ensure logrx works with quarto docs (qmd files) #201

Open
parmsam-pfizer opened this issue Aug 21, 2023 · 2 comments
Open

Feature Request: Ensure logrx works with quarto docs (qmd files) #201

parmsam-pfizer opened this issue Aug 21, 2023 · 2 comments
Assignees
@parmsam-pfizer
Labels
enhancement New feature or request

Comments

@parmsam-pfizer
Copy link
Collaborator

Feature Idea

As in #140 which ensure logrx works with rmd files, ensure logrx works with qmd files.

Relevant Input

No response

Relevant Output

No response

Reproducible Example/Pseudo Code

No response
@parmsam-pfizer parmsam-pfizer added the enhancement New feature or request label Aug 21, 2023
@parmsam-pfizer parmsam-pfizer mentioned this issue Aug 21, 2023
Merged
13 tasks
@bms63 bms63 moved this to 📋 Backlog in logrx 0.3.0 Sep 22, 2023
@parmsam-pfizer
Copy link
Collaborator Author

parmsam-pfizer commented Jan 16, 2025 •
edited
Loading

We use the envir parameter in rmarkdown::render(). quarto::quarto_render() doesn't have this parameter. This issue from the quarto R package Github repo goes into more detail: quarto-dev/quarto-r#73 (comment) Basically, every R code is run in a clean new session with Quarto.

@kodesiba kodesiba removed this from logrx 0.3.0 Jan 16, 2025
@kodesiba
Copy link
Collaborator

Add note near RMD files that Quarto will not be supported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@parmsam-pfizer parmsam-pfizer

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kodesiba @parmsam-pfizer