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

(CONT-670) Add legacy facts check #91

Merged
merged 2 commits into from
Feb 27, 2023
Merged

Conversation

chelnak
Copy link

@chelnak chelnak commented Feb 27, 2023

This change adds the Legacy Facts check originally authored by @mmckinst.

Additional contributions to the check were made by @seanmil, @rodjek, @baurmatt, @bart2 and @joshcooper

@chelnak chelnak requested a review from a team as a code owner February 27, 2023 11:29
@chelnak chelnak self-assigned this Feb 27, 2023
@chelnak chelnak added the enhancement New feature or request label Feb 27, 2023
This commit adds the Legacy Facts check originally authored by
@mmckinst.

Additional contributions to the check were made by @seanmil,
@rodjek, @baurmatt, @bart2 and @joshcooper
This commit adds the required spec tests for the legacy facts check.
@chelnak chelnak force-pushed the CONT-670-add_legacy_facts_check branch from 179b889 to b3d4e1c Compare February 27, 2023 12:06
Copy link

@GSPatton GSPatton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. Ran locally and works as expected

@GSPatton GSPatton merged commit 742187e into main Feb 27, 2023
@GSPatton GSPatton deleted the CONT-670-add_legacy_facts_check branch February 27, 2023 12:29
bastelfreak added a commit to bastelfreak/voxpupuli-puppet-lint-plugins that referenced this pull request Feb 28, 2023
puppet-lint-top_scope_facts-check got vendored into puppet-lint 3.1:
puppetlabs/puppet-lint#85

puppet-lint-legacy_facts-check got vendored into puppet-lint 3.1:
puppetlabs/puppet-lint#91
bastelfreak added a commit to bastelfreak/voxpupuli-puppet-lint-plugins that referenced this pull request Feb 28, 2023
puppet-lint-top_scope_facts-check got vendored into puppet-lint 3.1:
puppetlabs/puppet-lint#85

puppet-lint-legacy_facts-check got vendored into puppet-lint 3.1:
puppetlabs/puppet-lint#91
bastelfreak added a commit to bastelfreak/voxpupuli-puppet-lint-plugins that referenced this pull request Feb 28, 2023
puppet-lint-top_scope_facts-check got vendored into puppet-lint 3.1:
puppetlabs/puppet-lint#85

puppet-lint-legacy_facts-check got vendored into puppet-lint 3.1:
puppetlabs/puppet-lint#91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants