[Catalyst] Doc patch for ConfigLoader.
Tomas Doran
bobtfish at bobtfish.net
Thu May 14 09:25:28 GMT 2009
Louis Erickson wrote:
> I wrote a blog entry the other day about ConfigLoader and what the file
> names generated by MYAPP_CONFIG_LOCAL_SUFFIX etc. are. t0m suggested I
> write a patch. I've done so.
Nice one!
> Having done that, I am confident I used the most current code. =)
Applied perfectly, passes tests, looks good to me - applied as r10153
with Changelog entry.
> I also began to see why it's hard to catch these, as I noticed that this
> behavior is documented in the existing docs if you understand how it works
> already.
Yes, exactly!! That's why we try so hard to get people to write it down
when they have that lightbulb moment. It's impossible to document it
right when you just wrote the feature...
> With these docs in place, I would have found what I was looking for and
> not spent time debugging.
>
> Please let me know if you have any suggestions, changes, or if the patch
> does not work for some reason.
That was perfect.
Nice work, many many thanks.
t0m
More information about the Catalyst
mailing list