Merge pull request #4 from rameel/cleanup

Upgrade Ramstack.Parsing to 1.0.0-beta.7

Author: rameel

.github/workflows/publish.yml

@@ -24,7 +24,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Build Solution
-        run: dotnet build -c Release -p:SkipInternalsVisibleTo=true
+        run: dotnet build -c Release
 
       - name: Create NuGet Packages
         run: dotnet pack -c Release -o ./nuget --no-build

Directory.Packages.props

@@ -9,6 +9,6 @@
     <PackageVersion Include="MinVer" Version="6.0.0" />
     <PackageVersion Include="NUnit" Version="4.3.2" />
     <PackageVersion Include="NUnit3TestAdapter" Version="4.6.0" />
-    <PackageVersion Include="Ramstack.Parsing" Version="1.0.0-beta.5" />
+    <PackageVersion Include="Ramstack.Parsing" Version="1.0.0-beta.7" />
   </ItemGroup>
-</Project>
+</Project>

src/Ramstack.ExpressionParser/ExpressionBuilder.cs

@@ -277,7 +277,7 @@ Expression ProcessMemberAccess(Expr.MemberAccess e)
                         Error.NonInvocableMember(m);
 
                     //
-                    // It's not supposed to happen
+                    // This shouldn't happen, but we'll crash if it does
                     //
                     if (expression is null)
                         throw new InvalidOperationException(