Skip to content

Commit ef4678c

Browse files
authored
Upgrade dev analyzers to 1.0.0-alpha.18; make WindowsFsyncSupport internal, #1100 (#1202)
* Upgrade dev analyzers to 1.0.0-alpha.18; make WindowsFsyncSupport internal, #1100 * Suppress build warnings for LuceneDev1005 in test projects, and LuceneDev1006 in all projects since we can't fix yet
1 parent 6ed2050 commit ef4678c

4 files changed

Lines changed: 7 additions & 2 deletions

File tree

.build/dependencies.props

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
<!-- J2N will break binary compatibility in 3.0.0 to fix the APIs of collection types -->
4141
<J2NPackageVersion>[2.2.0-alpha-0021, 3.0.0)</J2NPackageVersion>
4242
<LiquidTestReportsMarkdownPackageVersion>1.0.9</LiquidTestReportsMarkdownPackageVersion>
43-
<LuceneNetCodeAnalysisDevPackageVersion>1.0.0-alpha.6</LuceneNetCodeAnalysisDevPackageVersion>
43+
<LuceneNetCodeAnalysisDevPackageVersion>1.0.0-alpha.18</LuceneNetCodeAnalysisDevPackageVersion>
4444
<MicrosoftAspNetCoreHttpAbstractionsPackageVersion>2.3.0</MicrosoftAspNetCoreHttpAbstractionsPackageVersion>
4545
<MicrosoftAspNetCoreTestHostPackageVersion>8.0.19</MicrosoftAspNetCoreTestHostPackageVersion>
4646
<MicrosoftBclMemoryPackageVersion>10.0.0-rc.1.25451.107</MicrosoftBclMemoryPackageVersion>

Directory.Build.targets

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,8 @@
3030
<NoWarn Label="Floating point type arithmetic needs to be checked on x86 in .NET Framework and may require extra casting">$(NoWarn);LuceneDev1002</NoWarn>
3131
<NoWarn Label="Java array parameters can sometimes be replaced with .NET out or ref parameters">$(NoWarn);LuceneDev1003</NoWarn>
3232
<NoWarn Label="Return array may be able to be replaced with out values">$(NoWarn);LuceneDev1004</NoWarn>
33+
<NoWarn Label="Floating point values embedded in strings should be formatted with J2N.Numerics Single or Double ToString methods">$(NoWarn);LuceneDev1006</NoWarn>
34+
3335
</PropertyGroup>
3436

3537
<PropertyGroup Label="Solution-level Publish to Project-specific Directory">

TestTargetFramework.props

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,9 @@
9191

9292
<!-- SonarCloud issues -->
9393
<NoWarn Label="Add at least one assertion to this test case">$(NoWarn);S2699</NoWarn>
94+
95+
<!-- Lucene.NET Dev Analyzers -->
96+
<NoWarn Label="Types in the Lucene.Net.Support namespace should not be public">$(NoWarn);LuceneDev1005</NoWarn>
9497
</PropertyGroup>
9598

9699
</Project>

src/Lucene.Net/Support/IO/WindowsFsyncSupport.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ namespace Lucene.Net.Support.IO
2222
* limitations under the License.
2323
*/
2424

25-
public static class WindowsFsyncSupport
25+
internal static class WindowsFsyncSupport
2626
{
2727
public static void Fsync(string path, bool isDir)
2828
{

0 commit comments

Comments
 (0)