traverse_and_update/3 return typespec needs html_tree() about floki

[philss]

@WLSF I think I can't assign directly to you. But feel free to work on this :)

[philss]

@philss yeah, I was wrong. It should be html_node() | html_tree() as @hattmarris suggested.
And yeah, you could replace that part by html_node() 👍. Thanks for verifying it :)

[philss]

@hattmarris I think you are right. The return type should be {html_tree(), traverse_acc} as it was before. I don't know why I didn't catch that, but I think the correct is the previous version.

Would you mind to send a pull request?

[WLSF]

@philss couple of things that I've noticed:

The function traverse_and_update/3 actually accepts a string as arg/response, as you can see on this test, is there a reason for it?

If thats by design then I guess that there are other aspects of this spec that might not be right, because right now its expecting the first arg and the response to be [html_node()], but as the test shows, its working with a simple string.

Assuming everything is ok with the STRING thing, I guess we could also replace this part by: html_node() since it already contains everything @type html_node :: html_tag() | html_comment() | html_doctype() | html_declaration() | html_text()