Skip to content

fix bug with setting no_managed_code #434

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

Merged
merged 4 commits into from
Sep 29, 2023

Conversation

EasyAsABC123
Copy link
Contributor

Signed-off-by jmschu02@gmail.com

Description
had to recreate
Fixes a bug in iis_pool that causes an issue with resetting to no_managed_code runtime versions

Issues Resolved
#395

Check List
All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
New functionality includes testing.
New functionality has been documented in the README if applicable
All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@EasyAsABC123
Copy link
Contributor Author

@tas50 did the 2012 images get removed because of deprecation? https://ci.appveyor.com/project/ChefWindowsCookbooks/iis/builds/19405861#L729

@tas50
Copy link
Contributor

tas50 commented Oct 10, 2018

I know AppVeyor changes their images from time to time. Sometimes we can specify and old image but they eventually remove even the old ones.

@EasyAsABC123
Copy link
Contributor Author

@tas50 looking at it actually makes no sense the .kitchen.appveyor.yml file only has

windows-2012R2

and it ran a couple of the tests...confusion abound. i'm sick and recently back from Germany so my brain is maybe at half function.

Signed-off-by: Justin Schuhmann <jmschu02@gmail.com>
Signed-off-by: Justin Schuhmann <jmschu02@gmail.com>
@EasyAsABC123 EasyAsABC123 requested a review from a team September 15, 2020 20:24
@xorima xorima merged commit c909d28 into sous-chefs:main Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium Will bring visible benefit to the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants