Skip to content
This repository has been archived by the owner on Nov 16, 2020. It is now read-only.

Add dump tag #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add dump tag #40

wants to merge 1 commit into from

Conversation

code28
Copy link

@code28 code28 commented Oct 23, 2018

Sometimes, it's good to know some more information about the variables inside the templates. Therefore, I considered a "dump"-tag useful. With my code, I wanted to introduce this concept. Maybe there is something better than debugPrint - I'm open for your feedback and input :)

Copy link
Member

@tanner0101 tanner0101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this addition! Just some small changes requested. Thanks!

Sources/TemplateKit/Tag/Dump.swift Show resolved Hide resolved
Sources/TemplateKit/Tag/Dump.swift Show resolved Hide resolved
Sources/TemplateKit/Tag/Dump.swift Outdated Show resolved Hide resolved
@tanner0101 tanner0101 added the enhancement New feature or request label Oct 30, 2018
@code28
Copy link
Author

code28 commented Oct 31, 2018

Thanks for your feedback! I changed, what you suggested.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants