Skip to content

Conversation

@schinckel
Copy link

No description provided.

@schinckel
Copy link
Author

Hmm. This doesn't quite work. More to do!

@schinckel
Copy link
Author

(It worked with my test project, but that was because I didn't have any template coverage in my tests ;)

@jambonrose
Copy link
Collaborator

jambonrose commented Sep 4, 2018

Thanks very much for the PR!

Several people have proposed alternative fixes to this issue. While this is a solid option and would work great, I'm going to opt for one of the other solutions. I hope this won't deter you from proposing changes in the future - your contributions are welcome!

@jambonrose jambonrose closed this Sep 4, 2018
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