Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove duplicate default factset files #77

Closed
glennsarti opened this issue Mar 26, 2018 · 1 comment
Closed

Remove duplicate default factset files #77

glennsarti opened this issue Mar 26, 2018 · 1 comment

Comments

@glennsarti
Copy link
Contributor

There are many duplicate files in the default fact set which serve no purpose. These should be removed and tests added to ensure they can not be re-added again.

glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previously there were many default fact files which are just duplicates.  This
commit adds a test will performs a simple file hash comparison of all default
factset files and fails and displays any duplicate factsets.

This test is initially skipped while the duplicate files are removed in later
commits.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 11.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 10.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 10.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previously there were many default fact files which are just duplicates.  This
commit adds a test will performs a simple file hash comparison of all default
factset files and fails and displays any duplicate factsets.

This test is initially skipped while the duplicate files are removed in later
commits.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 11.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 10.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previously there were many default fact files which are just duplicates.  This
commit adds a test will performs a simple file hash comparison of all default
factset files and fails and displays any duplicate factsets.

This test is initially skipped while the duplicate files are removed in later
commits.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 11.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 10.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previously there were many default fact files which are just duplicates.  This
commit adds a test will performs a simple file hash comparison of all default
factset files and fails and displays any duplicate factsets.

This test is initially skipped while the duplicate files are removed in later
commits.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 11.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
This commit removes the duplicate and redundant fact sets for Solaris 10.
glennsarti added a commit to glennsarti/facterdb that referenced this issue Mar 26, 2018
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
rodjek pushed a commit to glennsarti/facterdb that referenced this issue Jul 15, 2019
Previously there were many default fact files which are just duplicates.  This
commit adds a test will performs a simple file hash comparison of all default
factset files and fails and displays any duplicate factsets.

This test is initially skipped while the duplicate files are removed in later
commits.
rodjek pushed a commit to glennsarti/facterdb that referenced this issue Jul 15, 2019
This commit removes the duplicate and redundant fact sets for Solaris 11.
rodjek pushed a commit to glennsarti/facterdb that referenced this issue Jul 15, 2019
This commit removes the duplicate and redundant fact sets for Solaris 10.
rodjek pushed a commit to glennsarti/facterdb that referenced this issue Jul 15, 2019
Previosuly the test for unique factsets was created but skipped while the
duplicate files were removed.  Now that the files have been removed, this test
can be enforced.
rodjek added a commit that referenced this issue Jul 15, 2019
(#77)(#78)  Add default factset tests and fix failures
@rodjek
Copy link
Member

rodjek commented Jul 15, 2019

Fixed by #79

@rodjek rodjek closed this as completed Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants