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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for # inline comment tag #533

Merged
merged 1 commit into from
Jul 29, 2022
Merged

Conversation

charlespwd
Copy link
Contributor

@charlespwd charlespwd commented Dec 17, 2021

The # inline comment tag is coming. This PR adds support for it in theme-check.

Looking forward to using this 馃憤:

{% # theme-check-disable ParserBlockingJavaScript %}

Instead of this 馃憥 :

{% comment %}theme-check-disable ParserBlockingJavaScript{% endcomment %}

Currently blocked until Shopify/liquid#1498 is merged and we have a new Liquid release.

@charlespwd charlespwd changed the title Add support for upcoming new comment syntax Add support for # inline comment tag Dec 17, 2021
@charlespwd charlespwd added the blocked For when you can't work on something label Dec 17, 2021
@charlespwd charlespwd removed the blocked For when you can't work on something label Jun 3, 2022
@charlespwd charlespwd self-assigned this Jun 3, 2022
@ConduciveMammal
Copy link

@charlespwd this seems to have been pushed live now. Seems to work fine on my stores

@charlespwd
Copy link
Contributor Author

Hmm. Should probably bug them. It's still "unreleased" even though we use it in prod.

https://github.com/Shopify/liquid/blob/master/History.md#features

@blanklob
Copy link

I still have the issue.

@charlespwd charlespwd merged commit e748a71 into main Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants