Michael Go
1d97389fb0
render template name with Liquid Syntax errors
2023-03-02 16:50:55 -04:00
Michael Go and GitHub
3ff4170cb0
Merge pull request #1692 from Shopify/error-message-with-filepath
...
set context's template_name with template.name in render and include tag
2023-02-28 10:57:24 -04:00
Michael Go
24dceef552
set context's template_name with template's name
2023-02-28 10:55:38 -04:00
Michael Go
428c66ffac
add name attribute to Template for more flexibility to set context's template name
2023-02-27 17:11:52 -04:00
Michael Go
0fe4a5d144
render error message with actual template path
2023-02-27 11:01:21 -04:00
Dylan Thacker-Smith and GitHub
e650dc4195
Revert "Instrument usage of bug with iteration of String with offset or 0 limit ( #1667 )" ( #1677 )
...
This reverts commit c743936a78 .
2023-02-22 12:48:18 -05:00
Guilherme Carreiro and GitHub
9ab688eada
Update rubocop-shopify (2.7.0 -> 2.12.0) ( #1687 )
2023-02-15 08:40:59 +01:00
Erin Morrissey and GitHub
abef59d129
Merge pull request #1685 from Shopify/update-liquid-paths
...
Update link paths in docs markup
2023-02-14 10:37:15 -08:00
Erin Morrissey
940c3a4207
update link paths in docs markup
2023-02-14 10:34:15 -08:00
Guilherme Carreiro and GitHub
84a0289ebc
Fix CI (as it installs RuboCop 1.45.1 and fails) ( #1686 )
2023-02-14 12:24:38 +01:00
Michael Go and GitHub
4599e5459f
Merge pull request #1684 from Shopify/fix-variable-lookup-parse-timeout
...
fix variable lookup parse timing out with missing closing bracket
2023-02-02 10:39:46 -04:00
Dylan Thacker-Smith and GitHub
59c445f0e1
Avoid a couple of duplicate assertions ( #1675 )
2023-02-02 09:31:01 -05:00
Michael Go
bd9c3802c8
add variable parser timeout unit tests
2023-02-01 21:15:35 -04:00
Michael Go
2b40850e4a
fix variable lookup parse timing out with missing closing bracket
2023-02-01 19:52:30 -04:00
Marco Concetto Rudilosso and GitHub
22ded5f304
Merge pull request #1683 from Shopify/marco/raw-tag-whitespace-control
...
Allow raw tag to be used with whitespace control
2023-02-01 16:51:46 +00:00
Marco Concetto Rudilosso
ddc32b7bd8
actually test whitespace stripping
2023-02-01 15:52:34 +00:00
Marco Concetto Rudilosso
74e505f6fa
Allow raw tag to be used with whitespace control
2023-02-01 13:49:51 +00:00
Michael Go and GitHub
6a888d4564
Merge pull request #1680 from Shopify/recursively-parse-brackets
...
recursively parse brackets on variable lookup
2023-01-31 16:29:54 -04:00
Michael Go
dd257b3d66
add an unit test for double nest variable lookup
2023-01-30 20:36:14 -04:00
Michael Go
1aaf6ed019
recursively parse brackets on variable lookup
2023-01-30 20:19:35 -04:00
Michael Go and GitHub
daf93a83c2
Merge pull request #1676 from Shopify/tablerow-nil-params
...
raise invalid integer argument error from tablerow
2023-01-18 11:08:04 -04:00
Michael Go
e889a9da0b
use to_i to parse parameters of tablerow tag
2023-01-17 17:48:16 -04:00
Michael Go
0f11c97623
raise invalid integer argument error from tablerow
2023-01-16 18:53:53 -04:00
Dylan Thacker-Smith and GitHub
e804f36681
Merge pull request #1674 from Shopify/better-test-to-liquid-value
...
Improve test coverage of the to_liquid_value feature.
2023-01-16 10:44:41 -05:00
Dylan Thacker-Smith
619ed3fcd7
Add some additional to_liquid_value assertions
2023-01-13 15:46:12 -05:00
Dylan Thacker-Smith
cdb5cb06b2
Remove == method from drops from testing to_liquid_value
...
Since they could cause tests to pass without to_liquid_value being called
on the left side of the equality comparison.
2023-01-13 15:37:27 -05:00
Jean byroot Boussier and GitHub
bc153159e6
Merge pull request #1673 from Shopify/initialize-context
...
Initialize context to nil on the Drop class
2023-01-12 22:19:28 +01:00
Jemma Issroff
128b4e35be
Initialize context to nil on the Drop class
...
We believe this will reduce megamorphic exits in YJIT. There are
currently 241 separate shapes generated with edge_name "@context" in
SFR. Initializing it to nil will significantly reduce this number.
2023-01-12 15:35:22 -05:00
Dylan Thacker-Smith and GitHub
c743936a78
Instrument usage of bug with iteration of String with offset or 0 limit ( #1667 )
2023-01-11 14:05:50 -05:00
Dylan Thacker-Smith and GitHub
bf711a0521
Provide another assertion for to_liquid_value unless tag test ( #1672 )
...
Since the original one would pass even if to_liquid_value isn't called on
the BooleanDrop object.
2023-01-11 14:05:27 -05:00
Dylan Thacker-Smith and GitHub
e8731f27d9
Merge pull request #1671 from ashmaroli/use-builtin-bundle-cache
...
Use cache built into `ruby/setup-ruby` action
2023-01-11 11:40:11 -05:00
Ashwin Maroli
6a44c1ec77
Do not set ENV["BUNDLE_PATH"]
2023-01-11 22:06:47 +05:30
Ashwin Maroli
1beb87b446
Bump latest Ruby version in CI matrix
2023-01-11 20:58:17 +05:30
Ashwin Maroli
b839deb3a8
Use cache built into ruby/setup-ruby action
2023-01-11 20:50:44 +05:30
liamgriffin and GitHub
0b826120c0
Merge pull request #1665 from Shopify/update-description-divided_by-filter
...
[Liquid reference docs] Adding more content to divided_by filter
2023-01-09 14:12:23 +00:00
liamgriffin
936f803a4e
Removed trailing space
2023-01-06 11:36:20 +00:00
liamgriffin
5cd8a83fa6
Adding more content to divided_by filter
...
Adding more content to divided_by filter, by detailing the behaviour of how different results are generated by different types of divisors.
2023-01-06 11:19:13 +00:00
Dylan Thacker-Smith and GitHub
c2c6cb2b15
Allow commas to separate for tag attributes ( #1658 )
...
For consistency with tags like the `render` tag, where we actually
prefer to use commas to separate attributes
2022-11-29 09:00:55 -05:00
shainaraskas and GitHub
9ccbf64571
Merge pull request #1656 from Shopify/sr-inline-comment
...
clarify inline_comment
2022-11-14 10:38:55 -05:00
Shaina Raskas
8b1b9f649a
remove separate inline comment tag for IA reasons
2022-11-11 16:21:18 -05:00
Shaina Raskas
474315c6bb
edits
2022-11-11 16:03:52 -05:00
Shaina Raskas
667664bb22
clarify inline_comment
2022-11-11 15:34:35 -05:00
Dylan Thacker-Smith and GitHub
1cdd1f0834
refactor: Advance tokenizer array offset instead of using Array#shift ( #1653 )
...
Array#shift would move all the remaining elements of the array, which
is slower for larger arrays.
2022-11-09 13:03:33 -05:00
Dylan Thacker-Smith and GitHub
1cd5ec54f0
Merge pull request #1641 from Shopify/assert-template-result-static-env
...
Use static environment assert_template_result input values
2022-11-02 10:05:52 -04:00
Peter Zhu and GitHub
29732f4305
Merge pull request #1345 from Shopify/pz-fix-tags-in-comment
...
Fix tags in comment
2022-11-01 13:07:03 -04:00
Peter Zhu
abed47547c
Fix tags in comment
2022-11-01 13:04:41 -04:00
Dylan Thacker-Smith and GitHub
ace3fe15ac
Merge pull request #1648 from Shopify/fix-include-internal-error
...
Prevent an internal error in include tag from non-string template_name
2022-10-28 14:29:03 -04:00
Dylan Thacker-Smith
db8e85ab31
Prevent an internal error in include tag from non-string template_name
...
which would otherwise happen on `template_name.split('/')`
2022-10-28 13:59:00 -04:00
Dylan Thacker-Smith
f484b868d0
assert_template_result: Avoid using the BlankFileSystem
...
Since it doesn't reflect the real liquid usage that we are trying
to test against.
2022-10-28 13:55:33 -04:00
Dylan Thacker-Smith and GitHub
3e8994c258
Have subclasses of a tag inherit superclass's disabled tags ( #1646 )
2022-10-27 11:13:50 -04:00