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

Use internal copy of to_numeric() #191

Merged
merged 3 commits into from
Jul 22, 2022
Merged

Conversation

IndrajeetPatil
Copy link
Member

@IndrajeetPatil IndrajeetPatil commented Jul 22, 2022

Since it will be removed from datawizard: easystats/datawizard#197 (comment)

modelbased needs to be updated on CRAN before datawizard.

Also, easystats/easystats#255

Since it will be removed from datawizard: easystats/datawizard#197 (comment)

modelbased needs to be updated on CRAN before datawizard.
@IndrajeetPatil IndrajeetPatil changed the title Use internal copy of to_numeric Use internal copy of to_numeric() Jul 22, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 22, 2022

Codecov Report

Merging #191 (50ca343) into main (671ba59) will increase coverage by 0.18%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main     #191      +/-   ##
==========================================
+ Coverage   35.08%   35.27%   +0.18%     
==========================================
  Files          21       22       +1     
  Lines        1026     1029       +3     
==========================================
+ Hits          360      363       +3     
  Misses        666      666              
Impacted Files Coverage Δ
R/visualisation_recipe.estimate_grouplevel.R 0.00% <0.00%> (ø)
R/estimate_grouplevel.R 68.00% <100.00%> (ø)
R/utils.R 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

@IndrajeetPatil IndrajeetPatil merged commit 005e2ba into main Jul 22, 2022
@IndrajeetPatil IndrajeetPatil deleted the remove_datawizard_to_numeric branch July 22, 2022 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants