Skip to content

Take latest Master to Development #237

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 48 commits into from
Jun 2, 2015
Merged

Take latest Master to Development #237

merged 48 commits into from
Jun 2, 2015

Conversation

raghushantha
Copy link
Member

No description provided.

yutingc and others added 30 commits May 15, 2015 14:02
Take Appveyour changes to BugFixes
Move Build Status to top of page and make it light weight
Fix avoiduninitializedvariable not recognizing switchparameter
…yzer into BugFixes

Conflicts:
	Tests/Engine/GetScriptAnalyzerRule.tests.ps1
Add check for builtin variables for GlobalVar rule
…ignment

Fix use declared vars more than assignment
…word

Fix AvoidUsingPlainTextForPassword raised for types other than strings or object
yutingc and others added 18 commits May 26, 2015 15:23
Add check for both IsPlural and IsSingular for nouns
Update PowerShell capitalization.
Also remove AvoidTrapStatement rule and clean up all unnessary "using"
headers
Update namespace Powershell to PowerShell
…wordParameter

Fix parameters missing in a conditional statement of the rule AvoidUsingPlainTextForPasswordParameter
…FlowGraph

Fix a bug in the flow graphs for throw statements
…sion does not work for more than one variable
…reThanOneVariable

Simplifies logic of suppressing rule and fix a bug where rule suppression does not work for more than one variable
@yutingc
Copy link
Contributor

yutingc commented Jun 2, 2015

Looks good to me.

@raghushantha
Copy link
Member Author

Thanks Yuting. Will merge once Appveyor is green.

raghushantha added a commit that referenced this pull request Jun 2, 2015
Take latest Master to Development
@raghushantha raghushantha merged commit 6300b22 into development Jun 2, 2015
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