Chris McDonough <chr...@plope.com> added the comment: This looks almost right Sean. But it's possible (even probable) that the ISettings utility won't be registered when you manufacture a BfgLoader. So I think the call to _auto_reload needs to happen in get_source each time (or at least memoize its results on the first call of get_source).
Also, we try to test everything 100%. I've just added a "nosetests" section to repoze.bfg.jinja2's setup.cfg. If you install "nose" then run "python setup.py nosetests" within an updated copy of the repoze.bfg.jinja2 trunk, you'll see a coverage report,. Currently the trunk has 100% test coverage. With your patch it does not. We need to maintain the test coverage invariant to be able to accept the patch. If it's too much work, let me know and I can try to work it out myself. __________________________________ Repoze Bugs <b...@bugs.repoze.org> <http://bugs.repoze.org/issue88> __________________________________ _______________________________________________ Repoze-dev mailing list Repoze-dev@lists.repoze.org http://lists.repoze.org/listinfo/repoze-dev
