-
-
Notifications
You must be signed in to change notification settings - Fork 176
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
Add [folder] token to interpolateName #27
Conversation
Usure of whether this is the right way or not - will need to add more tests to confirm
@sokra Eager to get your feedback on this! I wasn't sure of the approach to take - I pondered doing |
Looks good, but I don't have time this week to merge it... Next week. |
@sokra Eagerly awaiting any updates you might have for this 😄 |
Thanks. Sorry for waiting so long. I was too busy. |
Awesome thanks for merging it in. 💃🏻🎉 Can't wait for it to be published! 👏🏼 |
Thank you @joshhunt! I am admittedly a terrible open source developer and never did get round to doing tests for this... but I look forward to using it now! |
Haha good hustle @joshhunt! |
#11 appears to be abandoned so I took another crack at it - this time with tests!
I noticed @mrm007 did some work (in #26) on writing tests for
interpolateName
, so I merged that in to extend from that. I would recommend his PR is merged before this one - once it is the diff here will more accurately show the work that I've done myself.Let me know if you have any feedback!