Message144515
> Sorry, it's been sitting at the bottom of my starred list for months,
> the prospect of reloading context always making it a little less
> attractive than something else.
I understand that very well :)
> Error was caused by an actual change in behavior
Ah, good, I had missed that change. I’ll just adapt the test to check that one-word == include.
I think I won’t add the Manifest._actions attribute as in your patch; it does not cost much to have the list once in manifest and another time in test_manifest.
The only change I’ll make (or maybe you’d like to do it?) before committing is to add a test to make sure the line marked with “this cannot happen” really cannot happen. |
|
Date |
User |
Action |
Args |
2011-09-25 01:54:20 | eric.araujo | set | recipients:
+ eric.araujo, tarek, alexis, jlove |
2011-09-25 01:54:20 | eric.araujo | set | messageid: <1316915660.39.0.903059245475.issue11751@psf.upfronthosting.co.za> |
2011-09-25 01:54:19 | eric.araujo | link | issue11751 messages |
2011-09-25 01:54:19 | eric.araujo | create | |
|