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

(PUP-11666) Mark resources type as valid for hosts and devices #8941

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

seanmil
Copy link
Contributor

@seanmil seanmil commented Oct 28, 2022

The resources type is useful on both hosts and devices and is compatible as is in both modes.

@seanmil seanmil requested a review from a team as a code owner October 28, 2022 13:26
@puppetlabs-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@seanmil seanmil changed the title Mark resources type as valid for hosts and devices (PUP-11666) Mark resources type as valid for hosts and devices Oct 28, 2022
@joshcooper
Copy link
Contributor

rspec-mocks published a new gem that broke puppet tests. You'll need to rebase on 95291a9 for tests to pass. Otherwise, change looks good.

@seanmil
Copy link
Contributor Author

seanmil commented Oct 31, 2022

@joshcooper Thanks! Rebased.

@CLAassistant
Copy link

CLAassistant commented Apr 19, 2023

CLA assistant check
All committers have signed the CLA.

@joshcooper
Copy link
Contributor

@seanmil could you rebase on 7.x if needed?

The resources type is useful on both hosts and devices and is
compatible as is in both modes.
@seanmil
Copy link
Contributor Author

seanmil commented Oct 25, 2023

@seanmil could you rebase on 7.x if needed?

@joshcooper Rebased from 7.x & Pushed. Do I need to open a new PR with a different target branch? 7.x?

@mhashizume mhashizume changed the base branch from main to 7.x October 25, 2023 17:23
@mhashizume
Copy link
Contributor

@seanmil I've edited your PR to target 7.x, I will close and reopen this PR to retrigger checks. Thank you!

@mhashizume mhashizume closed this Oct 25, 2023
@mhashizume mhashizume reopened this Oct 25, 2023
@mhashizume
Copy link
Contributor

Thanks for your contribution @seanmil !

@mhashizume mhashizume merged commit fec6702 into puppetlabs:7.x Oct 26, 2023
22 checks passed
@joshcooper joshcooper added the enhancement New feature or request label Nov 15, 2023
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.

5 participants