All committers have signed the CLA.

Thanks for the PR. I didn't really review, but if you want the CI to pass, you will have to run tests locally and accept snapshot changes. i.e. withtestthat::snapshot_accept() for us to see the direct implications of your changes.

Also, you have to change the expected digest value in test-as_raw_html.R` for tests to pass. This is normal when we change the html code of a table.

thanks! I've updated some of the testing code & accepted snapshots, so they should pass, but I didn't have a chance to reviewall the snapshot changes

@olivroy I've scanned through the snapshot results and addressed any issues I saw, so I think it's about set — at least no todos pending on my list for now.

oops, should've run check. removed the test dependency on stringr, and also fixed the regex in the helper in the last commit