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

Draf check_model vignette #574

Merged
merged 12 commits into from
May 10, 2023
Merged

Draf check_model vignette #574

merged 12 commits into from
May 10, 2023

Conversation

strengejacke
Copy link
Member

@strengejacke strengejacke commented Apr 8, 2023

Fixes #395

@codecov-commenter
Copy link

codecov-commenter commented Apr 8, 2023

Codecov Report

Merging #574 (2e96016) into main (a9dd8f7) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 2e96016 differs from pull request most recent head 65101c4. Consider uploading reports for the commit 65101c4 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main     #574      +/-   ##
==========================================
+ Coverage   48.77%   48.78%   +0.01%     
==========================================
  Files          84       84              
  Lines        5657     5657              
==========================================
+ Hits         2759     2760       +1     
+ Misses       2898     2897       -1     
Impacted Files Coverage Δ
R/check_collinearity.R 75.77% <ø> (ø)

... and 1 file with indirect coverage changes

@bwiernik
Copy link
Contributor

bwiernik commented Apr 9, 2023

Did you see the vignette branch?

@strengejacke
Copy link
Member Author

No... 🙈

@strengejacke
Copy link
Member Author

@bwiernik how do we want to proceed?

@bwiernik
Copy link
Contributor

I'll finish a draft on Saturday

@strengejacke
Copy link
Member Author

Brenton, I'll merge this for now, but we can either replace by, add or merge with your draft you're working on to avoid redundancies.

@strengejacke strengejacke merged commit c43ddec into main May 10, 2023
@strengejacke strengejacke deleted the checkmodel_vignette branch May 10, 2023 12:15
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.

Documentation for check_model()
3 participants