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

Add back code coverage workflow #145

Merged
merged 25 commits into from
Dec 14, 2023
Merged

Add back code coverage workflow #145

merged 25 commits into from
Dec 14, 2023

Conversation

mstackhouse
Copy link
Contributor

No description provided.

Alex Homel and others added 24 commits March 30, 2023 13:06
Rounding issue fixed to match SAS rounding approach
6835016
@asbates
account for 0 row data frame when getting denominator totals with a d…
8358f22 
…ifferent population data
@asbates
add test for bug fix #131
73ad52e
@mstackhouse
pull in copy of call_standardise
71e7732
@mstackhouse
Fix for #136
cc3b9dd
@mstackhouse
further fixes to fix unit tests for #136
eccdd6c
@mstackhouse
Docs for #138
83884b8
@mstackhouse
document fix
fc6f885
@mstackhouse
Switch up snapshots to use dataframes instead of tibbles for more con…
d438e54 
…sistent prints
@mstackhouse
snaps
cbddc85
@mstackhouse
snaps
039a762
@mstackhouse
Update snaps for dataframe print for better consistency
392a84f
@mstackhouse
Updates for snaps
063cfd6
@mstackhouse
Specifically fix and reverify #136, add supporting tests
aafec93
@mstackhouse
I think this is finally working and if it's not I really just don't k…
76e904c 
…now guys I really don't.
@mstackhouse
Add test coverage workflow
edf9b55
@mstackhouse
Merge pull request #144 from atorus-research/gh_actions_fix
e037427 
Add test coverage workflow
@mstackhouse
Update forcats version requirement
468319d
@mstackhouse
Add commenting to explain non-intuitive sections
c183c9b
@mstackhouse
remove dead and unnecessary code
c31a8e2
@elimillera
Push up new round test
3dca0aa
@mstackhouse
Merge pull request #139 from atorus-research/gh_issues_136_138
96911c4 
Fix Issues #136 and #138
@mstackhouse
Merge pull request #126 from atorus-research/gh_issue_124
0776d9c 
Gh issue 124
@mstackhouse
Merge branch 'devel' of github.com:atorus-research/Tplyr into 131-0-r…
daafa49 
…ow-bug

Bring branch up to date with devel
@codecov Codecov
Copy link

codecov bot commented Nov 7, 2023 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (main@004a8e0). Click here to learn what that means.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main     #145   +/-   ##
=======================================
  Coverage        ?   92.41%           
=======================================
  Files           ?       42           
  Lines           ?     2744           
  Branches        ?        0           
=======================================
  Hits            ?     2536           
  Misses          ?      208           
  Partials        ?        0

@asbates asbates merged commit eb1b6de into main Dec 14, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asbates asbates asbates 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
@mstackhouse @asbates @elimillera