-
Notifications
You must be signed in to change notification settings - Fork 8
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
combine_checkboxes #196
Merged
Merged
combine_checkboxes #196
Changes from 35 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
9b0471b
Reduce function initial draft
859926f
Reduce function fixes
c42399d
Small fixes
8862ac1
Draft tests, add no_val param
96c3309
Add keep param
218fca4
Fix `keep` param
54e3a99
Update documentation and API
ec5c19d
Update combine_checkbox api and docs
6b1fb48
Add check for if no fields exist in selection
d55dc00
Add check_fields_are_checkboxes function
261342d
Minor cleaning
e1d4eb8
Update version, test recheck workflow
7207f09
Test recheck workflow file
4f861e1
Fix linting
eb11152
Add combine_checkboxes() to pkgdown
7348324
Remove revdepcheck, update renv
62080af
Add standard checks for params
522d01d
Filename update
3a395cf
Filename change
347d2a3
Rename test file
rsh52 cce0d12
Fix record_id_field assign, remove rowwise call
rsh52 c250eda
Remove instrument_identifiers, use bind_cols
rsh52 b0a8564
Implement parse_labels, clean code, fix tests
rsh52 21f8879
Remove record_id field, lint
rsh52 31797c6
Apply additional cleanup suggestions
rsh52 2dfac9a
Add extract_metadata fnctn, tests
rsh52 ed55292
Support multiple values_to, logicals, new checks
c0b3885
Linting
7789a22
Update API, clean up, new methods, new docs
c185e39
Add check_metadata_fields_exist, update details
abdc512
Consoldiate and rework checkbox value conversion
50d47d6
Add names_repair strategy support
a6d150d
Remove names_suffix, restructure prefix/sep
0f868b8
Add names_glue spec
abefbee
Add glue support with names_glue
06d1337
Make glue dependency, remove install check
rsh52 dcb1029
Update glue spec handling
rsh52 0295650
check_equal_col_summaries() implementation
rsh52 127dd46
Update error message check_equal_col_summaries()
rsh52 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
on: | ||
workflow_dispatch: | ||
inputs: | ||
which: | ||
type: choice | ||
description: Which dependents to check | ||
options: | ||
- strong | ||
- most | ||
|
||
name: Reverse dependency check | ||
|
||
jobs: | ||
revdep_check: | ||
name: Reverse check ${{ inputs.which }} dependents | ||
uses: r-devel/recheck/.github/workflows/recheck.yml@v1 | ||
with: | ||
which: ${{ inputs.which }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
Package: REDCapTidieR | ||
Type: Package | ||
Title: Extract 'REDCap' Databases into Tidy 'Tibble's | ||
Version: 1.1.1 | ||
Version: 1.2.0 | ||
Authors@R: c( | ||
person("Richard", "Hanna", , "[email protected]", role = c("aut", "cre"), | ||
comment = c(ORCID = "0009-0005-6496-8154")), | ||
|
@@ -38,6 +38,7 @@ Imports: | |
stats | ||
Suggests: | ||
covr, | ||
glue, | ||
knitr, | ||
labelled, | ||
lintr, | ||
|
@@ -52,5 +53,5 @@ Config/testthat/edition: 3 | |
Encoding: UTF-8 | ||
Language: en-US | ||
LazyData: true | ||
RoxygenNote: 7.3.1 | ||
RoxygenNote: 7.3.2 | ||
Roxygen: list(markdown = TRUE) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,5 @@ | ||
# REDCapTidieR 1.2.0 | ||
|
||
# REDCapTidieR 1.1.1 (development version) | ||
|
||
Version 1.1.1 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can import
glue
. It's already a dependency of packages we import (tidyr
,stringr
at least) so we're not really changing anything by bumping it up fromSuggests