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


Reply via email to