Trevor McKeown

Results 16 comments of Trevor McKeown

jinja2.exceptions.TemplateSyntaxError: expected token 'end of print statement', got '_'

Hi, I am having a similar problem. It seems like jinja2 doesn't like one of the values of my json variables. I am also on Windows 10 WSL. Below is...

feat: show derived configuration

Saw a few nit picks. I might be able to refactor this patch to use fewer files, by naming the snapshots in the test cases. WIP.

feat: show derived configuration

Thanks @bcoe. I will make the changes above and have a new commit in the next week.

feat: show derived configuration

@bcoe, Please review again.

feat: show derived configuration

Thanks for your comments on these. Some of the PRs did need to get closed. Unfortunately, this one is a prerequisite for #518 at the moment.

Improve code quality by increasing code coverage: lib/source-map-from-file.js

@bcoe ready to go!

Improve code quality by increasing code coverage: lib/source-map-from-file.js

Completed another rebase after release.

Improve code quality by increasing code coverage: lib/source-map-from-file.js

Everything looks good except the tests on node 14. The CI process fails with the following error: ```#!/bin/bash Run npm ci --engine-strict npm ERR! Cannot read property '@bcoe/v8-coverage' of undefined...

Improve code quality by increasing code coverage: lib/source-map-from-file.js

I am going to pull this back into draft mode. I noticed another module for spies that looks much easier to implement.

Additional Configuration File Support

Reference to old PR (#436)