Skip to content

chore: modernize GeneticAlgorithm - nullable, SourceLink, LoggerLite 5, CI/CD/Sonar (v4.1.0)#1

Merged
PFalkowski merged 5 commits into
masterfrom
refresh/genetic-algorithm
Jun 16, 2026
Merged

chore: modernize GeneticAlgorithm - nullable, SourceLink, LoggerLite 5, CI/CD/Sonar (v4.1.0)#1
PFalkowski merged 5 commits into
masterfrom
refresh/genetic-algorithm

Conversation

@PFalkowski

Copy link
Copy Markdown
Owner

Summary

  • LangVersion latest, Nullable enable; net8.0 was already set
  • ILoggerLite -> ILogger (LoggerLite 5.0.0 API rename), LoggerLite 4.0.0 -> 5.0.0
  • Nullable annotations: ILogger?, FileInfo?, Genotype?, GenePool = null!
  • Add SourceLink 8.0.0, GenerateDocumentationFile, PackageReadmeFile; remove GeneratePackageOnBuild + stale PackageReleaseNotes; version -> 4.1.0
  • Add ci.yml, publish.yml (NuGet Trusted Publishing), sonar.yml workflows (target GeneticAlgorithm project only)
  • Rewrite README with CI/NuGet/SonarCloud badges and usage example

Test plan

  • dotnet build -- 0 warnings, 0 errors
  • Note: no automated test project in this repo

Generated with Claude Code

PFalkowski and others added 5 commits June 15, 2026 10:37
…/CD/Sonar (v4.1.0)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Avoids the SonarCloud 'Avoid expanding secrets in a run block' hotspot.
The secret is already mapped to the env var on each step.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Previously the sed replacement incorrectly changed the env mapping line
(SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}) to a bare var reference, or
the env block was missing entirely. The scanner received an empty/literal
token and authentication failed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@PFalkowski PFalkowski merged commit 6fb887f into master Jun 16, 2026
3 checks passed
@PFalkowski PFalkowski deleted the refresh/genetic-algorithm branch June 16, 2026 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant