coderabbitai · alexcoderabbitai · Jul 16, 2025 · Jul 16, 2025 · coderabbitai · Jul 16, 2025
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -0,0 +1,23 @@
+name: C# Linter
+
+# ───────────── TRIGGER ─────────────
+on:
+  pull_request:
+
+# ───────────── JOBS ─────────────
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+
+    - uses: actions/setup-dotnet@v4
+      with:
+        dotnet-version: '8.0.x'   # .NET 8 already contains 'dotnet format'; no extra tool install needed
+
+    # Restore & analyze the new project
+    - name: Restore packages
+      run: dotnet restore BadCodeProject.csproj
+
+    - name: Run dotnet format
+      run: dotnet format BadCodeProject.csproj --verify-no-changes --severity info
diff --git a/.github/workflows/resharper.yml b/.github/workflows/resharper.yml
@@ -0,0 +1,27 @@
+name: ReSharper Inspection
+
+on:
+  pull_request:
+
+jobs:
+  inspection:
+    runs-on: ubuntu-latest
+    name: ReSharper Code Inspection
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup .NET
+        id: setup-dotnet
+        uses: actions/setup-dotnet@v4
+        with:
+          dotnet-version: '8.0.x'
+
+      - name: Restore
+        run: dotnet restore BadCodeProject.sln
+
+      - name: Inspect code with ReSharper
+        uses: muno92/resharper_inspectcode@v1
+        with:
+          solutionPath: ./BadCodeProject.sln
+          dotnetVersion: ${{ steps.setup-dotnet.outputs.dotnet-version }} 
diff --git a/BadCode.cs b/BadCode.cs
@@ -0,0 +1,27 @@
+ //  Intentionally terrible C# to demonstrate linter failures
+
+
+using  System; using   System.Collections.Generic;   // extra spaces + wrong on-line ordering
+using System. Linq;                                   // misplaced space after “System.”
+using System.Threading.Tasks; // comment trailing spaces                                
-using  System; using   System.Collections.Generic;   // extra spaces + wrong on-line ordering
-using System. Linq;                                   // misplaced space after “System.”
-using System.Threading.Tasks; // comment trailing spaces                                
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
-using  System; using   System.Collections.Generic;   // extra spaces + wrong on-line ordering
-using System. Linq;                                   // misplaced space after “System.”
-using System.Threading.Tasks; // comment trailing spaces                                
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
+
+namespace BadlyFormattedNS   {     // brace on same line, double spaces
-namespace BadlyFormattedNS   {     // brace on same line, double spaces
+namespace BadlyFormattedNS
+{
-namespace BadlyFormattedNS   {     // brace on same line, double spaces
+namespace BadlyFormattedNS
+{
+public  class bad_class   // wrong casing + double spaces
-public  class bad_class   // wrong casing + double spaces
+public class BadClass
-public  class bad_class   // wrong casing + double spaces
+public class BadClass
+{
+           private static   readonly  int  ANSWER   =42;// field casing, spacing, rogue constant
+
+   public  BadCode( )   {Console . WriteLine( "Created" );}  // spaces everywhere & braces
-   public  BadCode( )   {Console . WriteLine( "Created" );}  // spaces everywhere & braces
+    public BadCode()
+    {
+        Console.WriteLine("Created");
+    }
-   public  BadCode( )   {Console . WriteLine( "Created" );}  // spaces everywhere & braces
+    public BadCode()
+    {
+        Console.WriteLine("Created");
+    }
+
+   public  void    DoStuff(   IDictionary< string ,List <int>> data    )
+       { if(data==null){throw new ArgumentNullException ( nameof ( data ) );}
+               foreach(  var kvp   in data  ){
+                Console.WriteLine(   $"Key:{kvp.Key}, Count = {kvp.Value.Count}"); }        /* crowded */
+       }  // incorrect indentation + brace style
-   public  void    DoStuff(   IDictionary< string ,List <int>> data    )
-       { if(data==null){throw new ArgumentNullException ( nameof ( data ) );}
-               foreach(  var kvp   in data  ){
-                Console.WriteLine(   $"Key:{kvp.Key}, Count = {kvp.Value.Count}"); }        /* crowded */
-       }  // incorrect indentation + brace style
+    public void DoStuff(IDictionary<string, List<int>> data)
+    {
+        if (data == null)
+        {
+            throw new ArgumentNullException(nameof(data));
+        }
+        
+        foreach (var kvp in data)
+        {
+            Console.WriteLine($"Key:{kvp.Key}, Count = {kvp.Value.Count}");
+        }
+    }
-   public  void    DoStuff(   IDictionary< string ,List <int>> data    )
-       { if(data==null){throw new ArgumentNullException ( nameof ( data ) );}
-               foreach(  var kvp   in data  ){
-                Console.WriteLine(   $"Key:{kvp.Key}, Count = {kvp.Value.Count}"); }        /* crowded */
-       }  // incorrect indentation + brace style
+    public void DoStuff(IDictionary<string, List<int>> data)
+    {
+        if (data == null)
+        {
+            throw new ArgumentNullException(nameof(data));
+        }
+        
+        foreach (var kvp in data)
+        {
+            Console.WriteLine($"Key:{kvp.Key}, Count = {kvp.Value.Count}");
+        }
+    }
+
+      internal  static async  Task  <   IEnumerable < int >   >GetValuesAsync(  )
+{
+           await Task . Delay (100 ) ;   // spaces around dot + inside parentheses
+           return   new List<int>{1 ,2,3 ,  4}; // inconsistent spacing
+}        // extra spaces before brace
+}
+} // No newline at end
diff --git a/BadCodeProject.csproj b/BadCodeProject.csproj
@@ -0,0 +1,13 @@
+<!-- BadCodeProject.csproj -->
+<Project Sdk="Microsoft.NET.Sdk">
+  <PropertyGroup>
+    <TargetFramework>net8.0</TargetFramework>
+    <Nullable>enable</Nullable>
+    <ImplicitUsings>enable</ImplicitUsings>
+  </PropertyGroup>
+
+  <!-- explicitly include the demo file so the project builds -->
+  <ItemGroup>
+    <Compile Include="BadCode.cs" />
+  </ItemGroup>
+</Project>
-<!-- BadCodeProject.csproj -->
-<Project Sdk="Microsoft.NET.Sdk">
-  <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
-    <Nullable>enable</Nullable>
-    <ImplicitUsings>enable</ImplicitUsings>
-  </PropertyGroup>
-
-  <!-- explicitly include the demo file so the project builds -->
-  <ItemGroup>
-    <Compile Include="BadCode.cs" />
-  </ItemGroup>
-</Project>
-<!-- BadCodeProject.csproj -->
-<Project Sdk="Microsoft.NET.Sdk">
-  <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
-    <Nullable>enable</Nullable>
-    <ImplicitUsings>enable</ImplicitUsings>
-  </PropertyGroup>
-
-  <!-- explicitly include the demo file so the project builds -->
-  <ItemGroup>
-    <Compile Include="BadCode.cs" />
-  </ItemGroup>
-</Project>
diff --git a/BadCodeProject.sln b/BadCodeProject.sln
@@ -0,0 +1,18 @@
+Microsoft Visual Studio Solution File, Format Version 12.00
+# Visual Studio Version 17
+VisualStudioVersion = 17.0.31903.59
+MinimumVisualStudioVersion = 10.0.40219.1
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "BadCodeProject", "BadCodeProject\BadCodeProject.csproj", "{12345678-1234-1234-1234-123456789012}"
+EndProject
+Global
+	GlobalSection(SolutionConfigurationPlatforms) = preSolution
+		Debug|Any CPU = Debug|Any CPU
+		Release|Any CPU = Release|Any CPU
+	EndGlobalSection
+	GlobalSection(ProjectConfigurationPlatforms) = postSolution
+		{12345678-1234-1234-1234-123456789012}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{12345678-1234-1234-1234-123456789012}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{12345678-1234-1234-1234-123456789012}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{12345678-1234-1234-1234-123456789012}.Release|Any CPU.Build.0 = Release|Any CPU
+	EndGlobalSection
+EndGlobal 
diff --git a/BadCodeProject/BadCodeProject.csproj b/BadCodeProject/BadCodeProject.csproj
@@ -0,0 +1,10 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <OutputType>Exe</OutputType>
+    <TargetFramework>net8.0</TargetFramework>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <Nullable>enable</Nullable>
+  </PropertyGroup>
+
+</Project> 
diff --git a/BadCodeProject/Program.cs b/BadCodeProject/Program.cs
@@ -0,0 +1,76 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+
+namespace BadCodeProject
+{
+    public class Program
+    {
+        public static void Main(string[] args)
+        {
+            // Unused variable
+            int unusedVariable = 42;
+
+            // Redundant type specification
+            List<string> strings = new List<string>();
+
+            // Possible null reference
+            string? nullableString = null;
+            Console.WriteLine(nullableString.Length);
+
+            // Inconsistent naming
+            int my_variable = 10;
+            int myVariable = 20;
+
+            // Unused parameter
+            void UnusedParameter(int unusedParam)
+            {
+                Console.WriteLine("Hello");
+            }
+
+            // Redundant parentheses
+            int result = (1 + (2 * 3));
+
+            // Magic number
+            if (result > 7)
+            {
+                Console.WriteLine("Too high");
+            }
+
+            // Inconsistent string literal
+            Console.WriteLine("Hello" + " " + "World");
+
+            // Unnecessary boxing
+            object boxed = 42;
+
+            // Redundant ToString() call
+            string number = 42.ToString();
+
+            // Unused method
+            void UnusedMethod()
+            {
+                Console.WriteLine("Never called");
+            }
+
+            // Possible multiple enumeration
+            var numbers = Enumerable.Range(1, 10);
+            var sum = numbers.Sum();
+            var count = numbers.Count();
+
+            // Inconsistent access modifiers
+            private int privateField = 0;
+            public int publicField = 0;
+
+            // Unnecessary cast
+            object obj = "string";
+            string str = (string)obj;
+
+            // Redundant conditional
+            bool condition = true;
+            if (condition == true)
+            {
+                Console.WriteLine("Redundant");
+            }
+        }
+    }
+}