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

Fixes infinite cycle #68

Closed
wants to merge 3 commits into from
Closed

Fixes infinite cycle #68

wants to merge 3 commits into from

Conversation

sanasol
Copy link

@sanasol sanasol commented Dec 10, 2019

when using non-stylesheet tag.

For example inserting Go-to-action that contains

<link itemprop="target" href="https://watch-movies.com/watch?movieId=abc123"/>

https://developers.google.com/gmail/markup/reference/go-to-action

@fedeisas
Copy link
Owner

fedeisas commented Apr 4, 2020

@sanasol thanks for this! This has been fixed on 7746a2f and will be released soon. Closing this.

@fedeisas fedeisas closed this Apr 4, 2020
@fedeisas fedeisas mentioned this pull request Apr 5, 2020
fedeisas added a commit that referenced this pull request Apr 5, 2020
fedeisas added a commit that referenced this pull request Apr 5, 2020
* Upgrade libraries and fix tests

* [#68] Ignoring non stylesheet link tags

* Composer normalize

* Tweak tests

* Composer normalize

* Remove Coversall

* Tests

* Cleanup

* Code style, fix typos, tweak tests
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.

2 participants