`tablerow` should be inserting newlines

[epage]

• for-loop code
• Ruby implementation
• Relevant tests
  • Once this is working, ideally the test will start failing. We would just need to remove the #[should_panic] on it
  • If the test doesn't start failing, please temporarily remove #[should_panic], and find or create an issue for the new failure

[Goncalerta]

Is this actually the case in ruby liquid?
I'm asking because I don't have ruby installed so I didn't actually try it with the original liquid. However, I tried with liquidjs and there were no newlines. I'm unsure if thats an incompatibility on their part (probably it is) or if the shopify documentation is misleading.

Either way, newlines increases readibility, so maybe its a good thing to have.

[epage]

I finished porting over all (relevant) liquid (ruby) integration tests to Rust (see conformance_ruby) and am still in the middle of annotating them with issues. At that point I'd update these with searches to all the failing tests.

Here are the relevant tests for this issue: https://github.com/cobalt-org/liquid-rust/blob/master/tests/conformance_ruby/tags/table_row_test.rs

[epage]

RE liquidjs

I guess technically this isn't needed for conformance since newlines are ignored in HTML and this tag exists to emit HTML but it makes it much easier to test conformance if we match up.