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.

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

169 wrapping up the long code in the Report #218

Merged
merged 45 commits into from
Oct 4, 2023
Merged
Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
6ebeb49
checking formatR and adding option to tidy options
Sep 26, 2023
007e574
updating news.md
Sep 27, 2023
0c5fdb1
fixing linter
Sep 27, 2023
26ea791
Merge 0c5fdb1db02de98bc2f09090d2aaac76d9fba092 into 4fa942ec6c8c72f46…
kartikeyakirar Sep 27, 2023
bf95d7e
[skip actions] Restyle files
github-actions[bot] Sep 27, 2023
0e7758b
Empty-Commit
Sep 27, 2023
33625e5
removing news
Sep 27, 2023
2dd832b
updating news and fixing linter issue.
Sep 28, 2023
c82fbb8
Merge branch 'main' into 169_wrap_rcode@main
kartikeyakirar Sep 28, 2023
2618a7f
fixing RMD check
Sep 28, 2023
69b6e7c
Merge branch '169_wrap_rcode@main' of https://github.com/insightsengi…
Sep 28, 2023
74cf1b9
exposing global_knitr parameter
Sep 29, 2023
79bd779
adding default options on functions efination.
Sep 29, 2023
a07f0f4
merge back from main
Sep 29, 2023
7bbb7f9
Merge a07f0f4224233e4dd3b5e3658ac60e68908b8693 into 4a2ee336480c9ff85…
kartikeyakirar Sep 29, 2023
9ca057a
[skip actions] Restyle files
github-actions[bot] Sep 29, 2023
8076344
setting teal.reporter.global_knitr.
Sep 29, 2023
684cfaa
Merge branch '169_wrap_rcode@main' of https://github.com/insightsengi…
Sep 29, 2023
d79df80
Merge 684cfaaba995cdec1fbcf81c6298f341b31a562e into 4a2ee336480c9ff85…
kartikeyakirar Sep 29, 2023
7e0e03c
[skip actions] Restyle files
github-actions[bot] Sep 29, 2023
7b06e93
Empty-Commit
Sep 29, 2023
a05d616
fixing lintr and added assertion on global_knitr
Sep 29, 2023
d379ddc
Removing the message from renderrmd and moving it onload()
Oct 2, 2023
5b0e811
adding startupMessage
Oct 2, 2023
07543fa
placing packageStartupMessage .onLoad()
kartikeyakirar Oct 2, 2023
54523a4
fixing cmd check
Oct 3, 2023
706d504
updated .onload()
kartikeyakirar Oct 3, 2023
8fa2e3a
Merge branch 'main' into 169_wrap_rcode@main
kartikeyakirar Oct 3, 2023
a08cca5
merge back from main and updated teal.reporter.global_knitr
kartikeyakirar Oct 3, 2023
cac16b0
Update R/Previewer.R
kartikeyakirar Oct 4, 2023
5f132ef
[skip actions] Roxygen Man Pages Auto Update
dependabot-preview[bot] Oct 4, 2023
b78d874
Updating documentation
kartikeyakirar Oct 4, 2023
577163f
Merge branch '169_wrap_rcode@main' of https://github.com/insightsengi…
kartikeyakirar Oct 4, 2023
1448099
Fixing linter
kartikeyakirar Oct 4, 2023
19a23f5
adding details for simpleReporter
kartikeyakirar Oct 4, 2023
4379685
Update R/DownloadModule.R
kartikeyakirar Oct 4, 2023
b1ab7c1
Update R/zzz.R
kartikeyakirar Oct 4, 2023
f08f5b8
Update R/DownloadModule.R
kartikeyakirar Oct 4, 2023
17bec72
[skip actions] Roxygen Man Pages Auto Update
dependabot-preview[bot] Oct 4, 2023
d222ef4
inherits details from simpleReporter
kartikeyakirar Oct 4, 2023
aef6767
empty commit
kartikeyakirar Oct 4, 2023
7b3b89c
use inline code for details and defined the min version of formatR
kartikeyakirar Oct 4, 2023
c5ed90a
Merge 7b3b89c9366f7501f785bd0e3f53a08b809a7061 into b82c2acb63cf2ef80…
kartikeyakirar Oct 4, 2023
bd4a839
[skip actions] Restyle files
github-actions[bot] Oct 4, 2023
71b6d8a
'Empty-Commit'
kartikeyakirar Oct 4, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ jobs:
with:
additional-env-vars: |
_R_CHECK_CRAN_INCOMING_REMOTE_=false
_R_CHECK_EXAMPLE_TIMING_THRESHOLD_=10
additional-r-cmd-check-params: --as-cran
enforce-note-blocklist: true
note-blocklist: |
Expand Down
3 changes: 2 additions & 1 deletion .lintr
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
linters: linters_with_defaults(
line_length_linter = line_length_linter(120),
cyclocomp_linter = NULL,
object_usage_linter = NULL
object_usage_linter = NULL,
indentation_linter = NULL
)
1 change: 1 addition & 0 deletions DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ Imports:
zip (>= 1.1.0)
Suggests:
DT (>= 0.13),
formatR,
Copy link
Contributor

Choose a reason for hiding this comment

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

hey @kartikeyakirar can you run verdepcheck flow to determine the minimal needed version of this package?
verdepcheck

ggplot2 (>= 3.4.0),
lattice (>= 0.18-4),
png,
Expand Down
2 changes: 2 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# teal.reporter 0.2.1.9004

* Supports automatic `Rcode` formatting using the suggested `formatR` package in reports.

# teal.reporter 0.2.1

### Miscellaneous
Expand Down
5 changes: 5 additions & 0 deletions R/DownloadModule.R
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,11 @@ download_report_button_srv <- function(id,
names(input_list) <- names(rmd_yaml_args)
global_knitr <- list()
if (is.logical(input$showrcode)) global_knitr <- list(echo = input$showrcode)
if (requireNamespace("formatR", quietly = TRUE)) {
global_knitr <- append(global_knitr, list(tidy.opts = list(width.cutoff = 60), tidy = TRUE))
} else {
message("For better code formatting, consider installing the formatR package.")
}
kartikeyakirar marked this conversation as resolved.
Show resolved Hide resolved
report_render_and_compress(reporter, input_list, global_knitr, file)
},
contentType = "application/zip"
Expand Down