Skip to content

feat: add startup configuration preflight - #33

Merged
kieronlanning merged 2 commits into
mainfrom
feat/startup-configuration-preflight
Oct 19, 2026
Merged

kieronlanning merged 2 commits into
mainfrom
feat/startup-configuration-preflight

Conversation

@kieronlanning

Copy link
Copy Markdown
Contributor

Summary

  • Adds ConfigurationPreflight, which runs before the pipeline is built and reports every missing requirement at once: an existing Build:Solution/Build:CSharpierPath, the local NuGet feed path, a NuGet credential or Trusted Publishing, and GITHUB_TOKEN/GITHUB_REPOSITORY_ID.
  • Release:Mode=LocalNuGet without LOCAL_NUGET_FEED_PATH now fails immediately instead of after restore/build/test/pack.
  • Publish and GitHub-release modules skip only on intent; a missing credential fails the run rather than silently skipping a requested release.
  • Disables the ModularPipelines ASCII logo (PipelineOptions.PrintLogo = false).
  • Documents the preflight, updates the skills, and unignores new source files under src/src/Build/.
  • Bumps package.json to 0.6.1.

Verification

  • dotnet build src/Build.slnx -c Release --warnaserror
  • just test - 435 tests passed
  • just release-matrix - 20/20; just config-matrix - 21/21
  • Manual: local NuGet run fails fast with the feed-path error; NuGet dry run succeeds with no credentials; a normal run has no logo.

- fail before any module when the local NuGet feed, NuGet credential, GitHub token/repository id, solution, or CSharpier path is missing

- report every missing requirement at once instead of after the build has run

- stop treating missing credentials as a module skip condition, so a requested release is never silently declined

- disable the ModularPipelines ASCII logo
@kieronlanning
kieronlanning merged commit 34a1c41 into main Oct 19, 2026
1 check passed
@kieronlanning
kieronlanning deleted the feat/startup-configuration-preflight branch October 19, 2026 10:15
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