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

Resolved problem with $Features due to Contains method being case sensit... #1

Merged
merged 1 commit into from
Apr 28, 2015
Merged

Conversation

mikefrobbins
Copy link
Contributor

...ive

The Contains method that is used throughout the Set-TargetResource
function is case sensitive. This caused a configuration to fail if the
features were specified in a case other than upper case which is how
they are specified in the function. The easiest way to resolve this
issue was to add a line to convert $Features to upper case.

…sitive

The Contains method that is used throughout the Set-TargetResource
function is case sensitive. This caused a configuration to fail if the
features were specified in a case other than upper case which is how
they are specified in the function. The easiest way to resolve this
issue was to add a line to convert $Features to upper case.
@msftclas
Copy link

Hi @mikefrobbins, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

vors added a commit that referenced this pull request Apr 28, 2015
Resolved problem with $Features due to Contains method being case sensit...
@vors vors merged commit 1fe7d18 into dsccommunity:dev Apr 28, 2015
NarineM pushed a commit that referenced this pull request Apr 26, 2016
narrieta pushed a commit that referenced this pull request Aug 2, 2016
t3mi referenced this pull request in t3mi/SqlServerDsc Jun 11, 2019
t3mi referenced this pull request in t3mi/SqlServerDsc Jun 12, 2019
t3mi referenced this pull request in t3mi/SqlServerDsc Jul 1, 2019
t3mi referenced this pull request in t3mi/SqlServerDsc Jul 1, 2019
@johlju johlju mentioned this pull request May 6, 2023
9 tasks
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

Successfully merging this pull request may close these issues.

3 participants