Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: feO2x/Light.GuardClauses
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v15.1.0
Choose a base ref
...
head repository: feO2x/Light.GuardClauses
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
  • 11 commits
  • 18 files changed
  • 2 contributors

Commits on Jul 24, 2026

  1. Configuration menu
    Copy the full SHA
    7c55515 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #177 from feO2x/optional-validation-during-source-…

    …code-transformation
    
    Skip validation in SourceCodeTransformation project
    feO2x authored Jul 24, 2026
    Configuration menu
    Copy the full SHA
    bd7325c View commit details
    Browse the repository at this point in the history
  3. docs: add AI plan to fix annotation cleanup during source export

    Signed-off-by: Kenny Pflug <[email protected]>
    feO2x committed Jul 24, 2026
    Configuration menu
    Copy the full SHA
    5982c3e View commit details
    Browse the repository at this point in the history
  4. fix: proper cleanup of annotations

    Signed-off-by: Kenny Pflug <[email protected]>
    feO2x committed Jul 24, 2026
    Configuration menu
    Copy the full SHA
    372ce05 View commit details
    Browse the repository at this point in the history
  5. 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
    feO2x and claude committed Jul 24, 2026
    Configuration menu
    Copy the full SHA
    484fcb7 View commit details
    Browse the repository at this point in the history
  6. ci: add verify generated single file job

    Signed-off-by: Kenny Pflug <[email protected]>
    feO2x committed Jul 24, 2026
    Configuration menu
    Copy the full SHA
    7dab9cd View commit details
    Browse the repository at this point in the history
  7. Merge pull request #180 from feO2x/179-source-export-annotation-cleanup

    Source export annotation cleanup
    feO2x authored Jul 24, 2026
    Configuration menu
    Copy the full SHA
    f8581a1 View commit details
    Browse the repository at this point in the history
  8. docs: add AI plan for bundled annotation type reachability

    Signed-off-by: Kenny Pflug <[email protected]>
    feO2x committed Jul 24, 2026
    Configuration menu
    Copy the full SHA
    a9b0baa View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    e6db5a3 View commit details
    Browse the repository at this point in the history
  10. 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]>
    feO2x and claude committed Jul 24, 2026
    Configuration menu
    Copy the full SHA
    cfd4f52 View commit details
    Browse the repository at this point in the history
  11. Merge pull request #182 from feO2x/181-bundled-annotation-type-reacha…

    …bility
    
    Bbundled annotation type reachability
    feO2x authored Jul 24, 2026
    Configuration menu
    Copy the full SHA
    7a52101 View commit details
    Browse the repository at this point in the history
Loading