Merge pull request #4612 from Inarion/Inarion-FirstSteps

Expands description of ShadowedDeclarationInspection

Author: Hosch250

Rubberduck.Core/Properties/Settings.Designer.cs

@@ -35,7 +35,7 @@
     <CodeInspection Name="EmptyModuleInspection" Severity="Hint" InspectionType="MaintainabilityAndReadabilityIssues" />
     <CodeInspection Name="LineLabelNotUsedInspection" Severity="Warning" InspectionType="CodeQualityIssues" />
     <CodeInspection Name="IntegerDataTypeInspection" Severity="Hint" InspectionType="CodeQualityIssues" />
-    <CodeInspection Name="ShadowedDeclarationInspection" Severity="DoNotShow" InspectionType="CodeQualityIssues" />
+    <CodeInspection Name="ShadowedDeclarationInspection" Severity="Warning" InspectionType="CodeQualityIssues" />
     <CodeInspection Name="ConstantNotUsedInspection" Severity="Warning" InspectionType="CodeQualityIssues" />
     <CodeInspection Name="DefaultProjectNameInspection" Severity="Suggestion" InspectionType="MaintainabilityAndReadabilityIssues" />
     <CodeInspection Name="EmptyCaseBlockInspection" Severity="Warning" InspectionType="MaintainabilityAndReadabilityIssues" />

Rubberduck.Core/Properties/Settings.settings

@@ -293,7 +293,7 @@ If the parameter can be null, ignore this inspection result; passing a null valu
     <value>An empty 'Loop' block without any executable statements, leaves a maintainer wondering about the intent of the code. Avoid writing code that doesn't need to be written.</value>
   </data>
   <data name="ShadowedDeclarationInspection" xml:space="preserve">
-    <value>Two declarations are in scope and have the same identifier name. This means that only one of them will be available to use.</value>
+    <value>Two declarations are in scope and have the same identifier name. Consider using fully qualified identifier names, otherwise only one of them will be available to use.</value>
   </data>
   <data name="ObsoleteErrorSyntaxInspection" xml:space="preserve">
     <value>The 'Error' statement only exists in the language to support legacy code that required it; prefer using 'Err.Raise' instead.</value>

Rubberduck.Resources/Inspections/InspectionInfo.resx

@@ -22,7 +22,7 @@ We're currently only using a tiny bit of this code editor's functionality (more
 
 ### [EasyHook](http://easyhook.github.io/index.html)
 
-Without the EasyHook library, many of our more advanced Unit Testing features would simply not be possible.  This library really lives up to its name, and allows us to intercept and inspect traffic through VBE7.dll and other unmanged libraries.
+Without the EasyHook library, many of our more advanced Unit Testing features would simply not be possible.  This library really lives up to its name, and allows us to intercept and inspect traffic through VBE7.dll and other unmanaged libraries.
 
 > EasyHook makes it possible to extend (via hooking) unmanaged code APIs with pure managed functions, from within a fully managed environment on 32- or 64-bit Windows XP SP2, Windows Vista x64, Windows Server 2008 x64, Windows 7, Windows 8.1, and Windows 10.