Skip to content

Next Release #224

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
Draft
wants to merge 124 commits into
base: master
Choose a base branch
from
Draft

Next Release #224

wants to merge 124 commits into from

Conversation

AARON-CLARK
Copy link
Collaborator

No description provided.

Jeff-Thompson12 and others added 25 commits December 15, 2022 14:48
@Jeff-Thompson12
Add function to create blockData object
281cc1a
@Jeff-Thompson12
Add wrappers for block data object
50a3d51
@Jeff-Thompson12
Allow passing of block data object to tg_gt()
4b97674
@Jeff-Thompson12
Export block data functions
31f4d48
@Jeff-Thompson12
Account for multiple data sets containing field
e84365d
@Jeff-Thompson12
Allow NA or NULL to be passed to dropdown
d9fd1e0
@Jeff-Thompson12
Set up the easier tables
a070413
@Jeff-Thompson12
Fix merge conflict with devel
e263d06 
Merge branch 'devel' into jt-152-step_one

# Conflicts:
#	inst/app/www/recipe.js
@Jeff-Thompson12
Change args to stat_arg in JSON
cc0eb85
@Jeff-Thompson12
Add S3 methods to parse recipes
ff959a2
@Jeff-Thompson12
Use recipes.json to populate pre-made tables
3b242dc
@Jeff-Thompson12
Move recipes.json to system file
7fcae3c
@Jeff-Thompson12
Repair format to RECIPES selectize input
5ae15d1
@Jeff-Thompson12
Import jsonlite::read_json()
3388b73
@Jeff-Thompson12
Add classes for filters applied to ADSL and ADAE for STAN tables
34e285a
@Jeff-Thompson12
Resolve merge conflict in recipe.js
1b93413 
Merge branch 'devel' into jt-152-step_one

# Conflicts:
#	inst/app/www/recipe.js
@Jeff-Thompson12
Try to keep for_gt() from triggering too soon
9350b5f
@Jeff-Thompson12
Clean up reactivity instead of using debounce()
f4836ff
@Jeff-Thompson12
Resolve merge conflicts with devel
b8beca8
@Jeff-Thompson12
Add tryCatch back in for validation on recipes
2455014
@Jeff-Thompson12
Fix broken recipe display message
ce5c00e
@Jeff-Thompson12
Fix tryCatch issue
4e24a3b
@Jeff-Thompson12
Update WORDLIST
67e9df4
@AARON-CLARK
Increment version number to 0.2.1.9000
f836dfb
@AARON-CLARK
move item in news file into appropriate location
c518f57
@AARON-CLARK AARON-CLARK changed the title Devel Next Release Mar 18, 2023
Jeff-Thompson12 and others added 23 commits April 21, 2023 09:52
@Jeff-Thompson12
Update text to explicitly tell the user to type their response
aa37926
@Jeff-Thompson12
Fix buggy messaging
1051824
@Jeff-Thompson12
Add a hanful more tests
b9be894
@Jeff-Thompson12
Update documentation for addBlock()
6896f88
@Jeff-Thompson12
Test specifying R version
8832279
@Jeff-Thompson12
Use renv R version for other Github Actions
3d94360
@Jeff-Thompson12
Correct inputs to Github Actions
15952c2
@Jeff-Thompson12
Don't output cat when running tests
0d0a3d4
@Jeff-Thompson12
Add couple basic tests for blockData
b542ca3
@Jeff-Thompson12
Remove {rcmdcheck} because it's already included in renv.lock
0c665e4
@Jeff-Thompson12
Use renv R version and packages for tests on devel
7bea3d2
@Jeff-Thompson12
Invalidate the cache
429d21d
@Jeff-Thompson12
Add cache back in
8e2baaa
@Jeff-Thompson12
Install gcc
53f1b5e
@Jeff-Thompson12
Oops
5eb4fdb
@Jeff-Thompson12
Don't use renv for devel check
418a6e6
@Jeff-Thompson12
Reset workflows to devel
c92465d
@Jeff-Thompson12
Update test-coverage
ccab09a
@Jeff-Thompson12
Update pkgdown
5db0a23
@Jeff-Thompson12
Update devel R-CMD check
7a8e25d
@Jeff-Thompson12
setup-renv not setup-r-renv
65e3326
@Jeff-Thompson12
Merge pull request #244 from Biogen-Inc/jt-243-fix_gha
5a37465 
Update Github Actions
@Jeff-Thompson12
Merge branch 'master' into devel
c10464a
@codecov
Copy link

codecov bot commented Apr 26, 2023 •
edited
Loading

Codecov Report

Merging #224 (c4088c1) into master (f41ed77) will increase coverage by 3.39%.
The diff coverage is 42.95%.

❗ Current head c4088c1 differs from pull request most recent head b373abf. Consider uploading reports for the commit b373abf to get more accurate results

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
+ Coverage   19.94%   23.34%   +3.39%     
==========================================
  Files          50       56       +6     
  Lines        4607     5038     +431     
==========================================
+ Hits          919     1176     +257     
- Misses       3688     3862     +174
Impacted Files Coverage Δ
R/mod_tableGen.R 0.00% <0.00%> (ø)
R/mod_tableGen_fct_filter_adae.R 0.00% <0.00%> (ø)
R/mod_tableGen_fct_filter_adsl.R 0.00% <0.00%> (ø)
R/mod_tableGen_fct_param_opts.R 0.00% <0.00%> (ø)
R/mod_tableGen_fct_recipe_incl.R 0.00% <0.00%> (ø)
R/mod_tableGen_ui.R 0.00% <0.00%> (ø)
R/mod_tableGen_utils.R 0.00% <0.00%> (ø)
R/run_app.R 0.00% <0.00%> (ø)
R/zzz.R 0.00% <0.00%> (ø)
R/blockData.R 68.92% <68.92%> (ø)
... and 1 more

... and 2 files with indirect coverage changes

Jeff-Thompson12 and others added 6 commits April 26, 2023 14:53
@Jeff-Thompson12
Merge branch 'devel' into jt126-blockdata
8bbc84d 
Merge branch 'devel' into jt126-blockdata
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@Jeff-Thompson12
Display group_by and title
b389210
@Jeff-Thompson12
Add label and label_source to blocks
ef757ee
@Jeff-Thompson12
Oops
8498ed8
@Jeff-Thompson12
Maybe update documents and not just roxygen
ebadc42
@AARON-CLARK
Merge pull request #145 from Biogen-Inc/jt126-blockdata
b373abf 
Function to help create `blockData` object
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AARON-CLARK @Jeff-Thompson12