Conversion extended#1
Merged
Merged
Conversation
options not as param but built in
# Conflicts: # src/Tests/ExistAll.SimpleConfig.UnitTests/ConfigBuilderWithConfigurationTests.cs # src/Tests/ExistAll.SimpleConfig.UnitTests/UnitTest1.cs
guy-lud
added a commit
that referenced
this pull request
Jul 14, 2026
* docs(03): capture phase context * docs(state): record phase 3 context session * docs(03): research public-surface/packaging/binder cleanup phase * docs: refresh session handoff (Phase 3 — discuss+research done, planner next) * docs(phase-3): add validation strategy * docs(03): create phase plan (public surface, packaging & binder cleanup) * docs(03): refine D-05 (split exe-skip by entry point) + resolve open questions * docs(03): finalize phase 3 plan (2 plans) after plan-check + architect/security/perf review * refactor(03-01): make SettingsHolder internal sealed (API-01/D-01) - Flip public class SettingsHolder to internal sealed - Removes an internal DTO from the public API surface - Reachable from tests/benchmark via existing InternalsVisibleTo * chore(03-01): remove dead Core.AspNet package (PKG-01/D-02) - Drop Core.AspNet Project entry from SimpleSettings.slnx - Drop Core.AspNet ProjectReference from UnitTests.csproj - Delete src/Core/ExistForAll.SimpleSettings.Core.AspNet/ directory - Test-local Core/AspNet/Environments.cs duplicate untouched - Published 2.0.0-alpha.0.* alphas: unlist is owner-optional follow-up * chore(03-01): float Microsoft.Extensions.* floor per-TFM (PKG-02/D-03) - Split four Microsoft.Extensions.* pins into per-TFM conditional ItemGroups - net8 floors: Configuration 8.0.0, Configuration.Json 8.0.1, DI 8.0.1, DI.Abstractions 8.0.2 - net10 stays 10.0.9 for all four - Frees net8 consumers of the packable Binders + GenericHost packages from 10.x * docs(03-01): complete public-surface/packaging/binder-cleanup plan * test(03-02): add failing CLI binder cases for lookahead + SkipFirstArgument - space-separated --k v, quoted-with-spaces single token - prefixed-next-token = new key, prefixed-value non-binding - SkipFirstArgument true/false, empty-token no-crash - CLI-path secret-redaction regression (S2) * feat(03-02): add SkipFirstArgument option + lookahead CLI parse (SRC-02/D-04,D-05) - SkipFirstArgument (default false, explicit override; no silent arg[0] drop) - Parse rewritten to index/lookahead loop; space-separated --k v binds - empty-safe zero-alloc prefix detection (length-delta + Array.IndexOf, cached prefix array) - prefixed next-token treated as new key; ArgumentPrefixes XML-doc note * feat(03-02): AddCommandLine sources GetCommandLineArgs + owns exe skip (SRC-02/Open Q #1) - replace Environment.CommandLine.Split(' ') with Environment.GetCommandLineArgs() - wrap caller action to enable SkipFirstArgument=true (exe skip owned by this entry point) - quoted-value-with-spaces criterion now holds end-to-end unconditionally * docs(03-02): complete command-line binder cleanup plan * fix(03-02): null-guard CLI lookahead value token (post-review hardening) * docs(phase-03): complete phase execution (verified 4/4, review clean) * docs(backlog): capture client pre-beta engine requirements (COLL/VAL/API)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.