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
Handle tables in data #528
Conversation
@@ -76,19 +76,62 @@ test_that("JSON output for list of matrices is correct", { | |||
file = test_path("answers", "json-matrix-lists.json")) | |||
}) | |||
|
|||
test_that("write_stan_json() throws correct errors", { |
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 split these tests up into separate tests for the different errors
Codecov Report
@@ Coverage Diff @@
## master #528 +/- ##
==========================================
+ Coverage 92.78% 92.79% +0.01%
==========================================
Files 12 12
Lines 3035 3041 +6
==========================================
+ Hits 2816 2822 +6
Misses 219 219
Continue to review full report at Codecov.
|
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.
Great! Thanks!
Submission Checklist
Summary
Closes #527 by removing the "table" class from tables.
Copyright and Licensing
Please list the copyright holder for the work you are submitting
(this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the following licenses: