-
Notifications
You must be signed in to change notification settings - Fork 9
Comparing changes
Open a pull request
base repository: feO2x/Light.GuardClauses
base: v15.1.0
head repository: feO2x/Light.GuardClauses
compare: main
- 11 commits
- 18 files changed
- 2 contributors
Commits on Jul 24, 2026
-
chore: add option to skip validation in SourceCodeTransformation project
Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7c55515 - Browse repository at this point
Copy the full SHA 7c55515View commit details -
Merge pull request #177 from feO2x/optional-validation-during-source-…
…code-transformation Skip validation in SourceCodeTransformation project
Configuration menu - View commit details
-
Copy full SHA for bd7325c - Browse repository at this point
Copy the full SHA bd7325cView commit details -
docs: add AI plan to fix annotation cleanup during source export
Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 5982c3e - Browse repository at this point
Copy the full SHA 5982c3eView commit details -
fix: proper cleanup of annotations
Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 372ce05 - Browse repository at this point
Copy the full SHA 372ce05View commit details -
fix: align comments that follow XML documentation during source export
NormalizeWhitespace indents a single-line comment by one additional character when it shares a trivia list with an XML documentation comment. Removing an attribute list merges these two into the same list, but the misalignment also occurred without annotation removal and was present in the committed single file. Align such comments with the XML documentation above them in the line-based formatting cleanup, and regenerate the single file. The regeneration also picks up the "using System.Threading" directive that became stale with MustBePositiveOrInfinite. Furthermore, drop the unreachable throw when determining the unqualified attribute name, make the alias-qualified test source compilable, and pass the test cancellation token when parsing. Signed-off-by: Kenny Pflug <[email protected]> Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01KVPxLqTavx5hjAFWHe8TMc
Configuration menu - View commit details
-
Copy full SHA for 484fcb7 - Browse repository at this point
Copy the full SHA 484fcb7View commit details -
ci: add verify generated single file job
Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7dab9cd - Browse repository at this point
Copy the full SHA 7dab9cdView commit details -
Merge pull request #180 from feO2x/179-source-export-annotation-cleanup
Source export annotation cleanup
Configuration menu - View commit details
-
Copy full SHA for f8581a1 - Browse repository at this point
Copy the full SHA f8581a1View commit details -
docs: add AI plan for bundled annotation type reachability
Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for a9b0baa - Browse repository at this point
Copy the full SHA a9b0baaView commit details -
chore: implement annotation reachability via Roslyn on Source Export
Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e6db5a3 - Browse repository at this point
Copy the full SHA e6db5a3View commit details -
fix: treat using aliases as reachability roots on Source Export
The annotation reachability trimmer established reachability from attribute usages only, but a using alias names its target type as well. Whenever no [NotNull] usage survived cleanup, the code-analysis NotNullAttribute polyfill was trimmed while the alias directive that names it remained, which does not compile - reproducible with a netstandard2.0 whitelist export that retains no annotated assertion. Bind the target of every alias directive and mark the bundled type it resolves to, using the same degraded-binding fallback as attribute usages, so a failed binding keeps too much rather than too little. Compile every export of the trimmer tests through the generated-file build validator: an over-eager trim removes a declaration that the generated file still names, which no assertion about emitted types can catch. The export that supplies the JetBrains annotations externally is the one exception because it cannot compile on its own. Group the test classes that run MSBuild on the shared source validation project into one collection so their builds no longer race for its intermediate output. Also resolve the exporter's metadata references through a Lazy, annotate TryGetBundledType with MaybeNullWhen, expose the bundled type catalog as read-only lists, and print the merger's progress in execution order. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> Signed-off-by: Kenny Pflug <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for cfd4f52 - Browse repository at this point
Copy the full SHA cfd4f52View commit details -
Merge pull request #182 from feO2x/181-bundled-annotation-type-reacha…
…bility Bbundled annotation type reachability
Configuration menu - View commit details
-
Copy full SHA for 7a52101 - Browse repository at this point
Copy the full SHA 7a52101View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v15.1.0...main