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

reproduce error in print_report() function #172

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

reproduce error in print_report() function #172

wants to merge 30 commits into from

Conversation

Karim-Mane
Copy link
Member

This PR is intended to replace #166.

Here, I have created this branch from commit eebe908.

Karim-Mane and others added 30 commits April 5, 2024 01:44
@Karim-Mane
coerce id column into character
d87e0e3
@Karim-Mane
rename some files and functions and use standardize
ed4c52c
@Karim-Mane
allow for using column names before column names standardisation is p…
8e93416 
…erformed
@actions-user
Automatic readme update
9ed144f
@Karim-Mane
remove unnecesary line in DESCRIPTION file
a97c504
@Karim-Mane
disable option for removing bad date sequences found by check_date_se…
207d829 
…quence()
@Karim-Mane
disable possibility to provide a comma-separated list of column names.
a85ce72
@Karim-Mane
remove the internal call of standardize_dates() from check_date_seque…
47821a3 
…nce()
@Karim-Mane
use || instead of |
4e46588
@Karim-Mane
optimize on the code in detect_to_numeric_columns() as suggested by Hugo
0792230
@Karim-Mane
use '.data$var' from rlang in some dplyr functions
6dafba4
@Karim-Mane
use regex to check prefix and suffix
f7595f8
@Karim-Mane
allow for multiple prefix and suffix
5ad6e62
@Karim-Mane
fix pkgdown issue
4c4aec9
@Karim-Mane
create function to set the default cleaning operations.
87eb266
@Karim-Mane
update clean_data() documentation and account for the default cleanin…
36917e8 
…g arguments.
@Karim-Mane
send warning the missing character is not found.
9f3f819
@Karim-Mane
update documentation for standardize_dates() function
05bd258
@Karim-Mane
fine check params argument in clean_data() and fix linters
84654b2
@Karim-Mane
added new line to separate between messages and warning/errors
2b02dd3
@Karim-Mane
allow for spelling check to run
b9bf764
@Karim-Mane
document how the common_na_strings data was generated
a270623
@Karim-Mane
fix typos in design vignette
65c0d49
@Karim-Mane
add the lang argument needed by convert_to_numeric()
e641dd2
@Karim-Mane
use a vector for the rename argument in standardize_column_names()
eac31c2
@Karim-Mane
reduce else in scan_columns()
23926f9
@Karim-Mane
update the way the keep argument is handle in standardize_column_names()
07f237a
@Karim-Mane
update replace_missing_values() with the usage of dplyr
b638d9a
@Karim-Mane
update add_to_report()
eebe908
@actions-user
Automatic readme update
248e9ce
@Karim-Mane Karim-Mane mentioned this pull request Aug 12, 2024
Closed
@avallecam avallecam mentioned this pull request Oct 24, 2024
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Degoot-AM Degoot-AM Degoot-AM approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Karim-Mane @Degoot-AM @actions-user