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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add support for the Jekyll push filter #611

Merged
merged 1 commit into from
May 23, 2023

Conversation

TomasHubelbauer
Copy link
Contributor

@TomasHubelbauer TomasHubelbauer changed the title Add support for the Jekyll push filter [feat]: Add support for the Jekyll push filter May 22, 2023
@TomasHubelbauer TomasHubelbauer changed the title [feat]: Add support for the Jekyll push filter feat: Add support for the Jekyll push filter May 22, 2023
@TomasHubelbauer
Copy link
Contributor Author

Note that this is lacking tests at the moment, I will add them soon.

@harttle harttle merged commit dca9ab5 into harttle:master May 23, 2023
@TomasHubelbauer
Copy link
Contributor Author

TomasHubelbauer commented May 23, 2023

Thanks for the merge! I didn't add tests yet :D I will add them in another PR soon!

{% assign arr = '' | split: '' %}
{% assign arr = arr | push: 'test' %}
{{ arr }}
{{ arr | size }}

This might actually be broken assuming the Playground is up to date now after the merge. @harttle can you double check on this?

Also what do I need to do to make the push filter name highlighted via the syntax highlighter?

@github-actions
Copy link

github-actions bot commented Jun 3, 2023

🎉 This PR is included in version 10.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants