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

Match allowed datatypes to yumrepo skip_if_unavailable support #672

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

seanmil
Copy link
Contributor

@seanmil seanmil commented Sep 27, 2023

In PR #663 an overly strict datatype was set for the skip_if_unavailable parameter. The yumrepo type supports a variety of string values and boolean values.

@seanmil seanmil requested review from bastelfreak and a team as code owners September 27, 2023 14:41
@seanmil
Copy link
Contributor Author

seanmil commented Sep 27, 2023

Hmm... unless I'm looking in the wrong spot the "PA" project is marked as archived and I cannot create issues there to reference in the commit message (per the error in Static Code Analysis check).

@mhashizume
Copy link
Contributor

Thanks for this @seanmil . Since this is just a minor correction, could you add "(maint)" to the beginning of your commit summary to get it to pass the static code analysis check?

@seanmil seanmil force-pushed the expand_skip_if_unavailable_type branch from 9f9a02a to 3c5206f Compare September 28, 2023 11:49
@mhashizume
Copy link
Contributor

Ah sorry one more thing, could you regenerate REFERENCE.md with your changes? You should be able to run bundle exec puppet strings generate --format markdown --out REFERENCE.md then add the updated REFERENCE.md to your commit.

Thanks again!

In PR puppetlabs#663 an overly strict datatype was set for the
skip_if_unavailable parameter. The yumrepo type supports
a variety of string values and boolean values.
@seanmil seanmil force-pushed the expand_skip_if_unavailable_type branch from 3c5206f to 7e3f826 Compare September 28, 2023 17:24
@mhashizume mhashizume added the maintenance Maintenance chores are typically excluded from changelogs label Sep 28, 2023
@mhashizume mhashizume merged commit 3663482 into puppetlabs:main Sep 28, 2023
17 checks passed
@seanmil seanmil deleted the expand_skip_if_unavailable_type branch September 29, 2023 11:35
@mhashizume mhashizume added bug Something isn't working and removed maintenance Maintenance chores are typically excluded from changelogs labels Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants