Re: AnyData patches

excellent, thankyou for the list

updates pushed to http://github.com/SvenDowideit/AnyData


(onlist is fine by me - others might have suggestions for me too :))

that sent me off to learn a pile more - though some was easier to 
address using Module::Build than MakeMaker (which AnyData is still using 
for now)


I (think) i've done all except the examples/ suggestion, which I might 
leave until i've fixed some more bugs and have more of an idea of what 
can be done using AnyData


Cheers

Sven


On 05/09/12 16:47, H.Merijn Brand wrote:
 > I have a *lot* of modules - for testing - installed, but still run into
 > new ones, and neither is mentioned when running Makefile.PL
 >
 > t/01distributtion.t .. skipped: Test::Distribution not installed
 > t/01pod_snippets.t ... Can't locate Test/Pod/Snippets.pm
 >
 > And Test::Pod::Snippets depends on a lot of unneeded stuff like
 > Object::InsideOut and Module::Locate, two I also had not installed.
 >
 > If you really want to depend on those just locally, disable these tests
 > in the distribution if the module cannot be loaded. Even after having
 > the modules installed, I get
 >
 > t/01distributtion.t .. skipped: Test::Distribution not installed
 > t/01pod_snippets.t ... skipped: Need to do more work fixing pod code 
snippets
 >
 > so I guess it is better to just drop that test and use Test::Pod and
 > Test::Pod::Coverage instead:

I've moved the author tests I have started using into xt/ - though 
unlinke in M:B I've not worked out how to activate them cleanly

I'm planning to use Test::Pod::Snippets, because it test the code 
examples in the Pod, but for now its not that happy

Test::Pod and Coverage are in xt/ now too :)


 >
 > I miss an examples folder. Docs are fine, but examples are great

This is a good idea for later - initially, I want to get some of the 
bugs fixed

 >
 > Makefile.PL requires 5.004. I'd say that by now it is safe to require
 > at least 5.6.2, but when also using DBI, require 5.8.1. Additional, you
 > can then add:

5.6 it is :) though clearly I need to tweak the docco to show AnyData 
doesn't use DBI

 >
 > * use_warnings
 > Add 'use warnings' to all modules. (This will require perl > 5.6)

done - I hope there aren't any lurking consequenses that I won't know 
about due to a lack of test cases

 >
 > MYMETA.json/yml should include the git repo under resources

META_ADD-ed

 >
 > Spell-checking - after filtering known words. I can give you the tools
 > on IRC
 >

xt/spelling.t :)

 >
 >
 > Not in MANIFEST: AnyData/Storage/File.pod
 > Not in MANIFEST: t/htmltable.t
 > Not in MANIFEST: t/xml.t

updated using make manifest

 >
 > * has_humanreadable_license
 > Add a section called 'LICENSE' to the documentation, or add a file 
named LICENSE to the distribution.

in M:B i used create_license - I didn't see the same in MM, so I copied 
a LICENSE file for now

 >
 > * has_test_pod_coverage
 > Add a test using Test::Pod::Coverage to check for POD coverage.
 >

done