reesemclean
Add current date/time template variables.
Adds simple support for inserting current date/time components into templates by variable substitution.
Sorry for the delay in taking a look... the concept looks good to me... could you add an integration test for this behavior so we can make sure it stays working.
Let me know if have question on adding a test.
I have some test cleanup changes as well, but I'll save that for a separate pull request unless you'd prefer I include it now.
If it’s unrelated, a separate one is fine... currently the tests are not succeeding on the CI so I need to figure that out as well.
On Dec 9, 2019, at 2:56 PM, Kurt Hardin [email protected] wrote: I have some test cleanup changes as well, but I'll save that for a separate pull request unless you'd prefer I include it now.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.
Ok, I'll open a separate request for those changes then since they are unrelated.
BTW, the CI error appears to be related to specifying a fairly old version of rvm in .travis.yml.
Unfortunately that is just the beginning of the issues - the vscode extension test example seems to have changed a bit so working through that. I can manually get this PR through for now though.
On Dec 9, 2019, at 3:16 PM, Kurt Hardin [email protected] wrote:
Ok, I'll open a separate request for those changes then since they are unrelated.
BTW, the CI error appears to be related to specifying a fairly old version of rvm in .travis.yml.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.
Got the extension test running on the CI again, can you resolve the conflicts and then I will merge in and release a new version. Thanks!
