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

Mojo::Template suggested update to documentation #230

Closed
wants to merge 1 commit into from
Closed

Mojo::Template suggested update to documentation #230

wants to merge 1 commit into from

Conversation

alabamapaul
Copy link

Adding section about Comments to inform the developer of potential pitfalls of tags within comments.

@kraih
Copy link
Member

kraih commented Oct 18, 2011

I wouldn't mind some clarification around the tokenizer situation, we've actually just talked about it on the IRC channel.
But i don't think this is the right way, since it affects all tags.

@kraih kraih closed this Oct 18, 2011
@alabamapaul
Copy link
Author

I am not certain how this affects all tags since it is a change to POD only.

Perhaps someone who has a better understanding of where documentation such as this should go could take a few minutes of their time to address this, or provide a suggestion as to where (within the documentation) something such as this should be mentioned, and I will gladly draft patches in the appropriate location.

I was concerned because I went from having a functioning Mojolicious application before upgrading to 2.0, to having an application that caused Exceptions after upgrading to 2.0, and wanted to provide documentation for others who may experience a similar situation.

For now, I guess others are at the mercy of finding this information using a search engine rather than within the documentation.

@kraih
Copy link
Member

kraih commented Oct 18, 2011

I meant that the explanation itself should be something less specific, perhaps a line somewhere about the nesting of tags. (as you said on the list)

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