Tux pointed out that no_symlinks should exclude files listed in MANIFEST.SKIP even when we are under developing environment (i.e. Test::Kwalitee), because they don't go into a distribution. There should be more tests that should respect MANIFEST.SKIP, or, maybe everything listed in MANIFEST.SKIP should be excluded before actual file testing.
When we implement this, it would also be better to have another test that looks at MANIFEST.SKIP to see if important files are not listed, so that Test::Kwalitee users don't get upset when they are told something is missing when they see it in the repository.
This needs to change the order of testing.
Tux pointed out that no_symlinks should exclude files listed in MANIFEST.SKIP even when we are under developing environment (i.e. Test::Kwalitee), because they don't go into a distribution. There should be more tests that should respect MANIFEST.SKIP, or, maybe everything listed in MANIFEST.SKIP should be excluded before actual file testing.
When we implement this, it would also be better to have another test that looks at MANIFEST.SKIP to see if important files are not listed, so that Test::Kwalitee users don't get upset when they are told something is missing when they see it in the repository.
This needs to change the order of testing.