You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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.
The text was updated successfully, but these errors were encountered: