Skip to content

Remove temporal comments. - #942

Closed
Gudge (MGudgin) wants to merge 2 commits into
user/gudge/version_specific_config_parsers_phase5dfrom
user/gudge/version_specific_config_parsers_phase5d.1
Closed

Remove temporal comments.#942
Gudge (MGudgin) wants to merge 2 commits into
user/gudge/version_specific_config_parsers_phase5dfrom
user/gudge/version_specific_config_parsers_phase5d.1

Conversation

@MGudgin

@MGudgin Gudge (MGudgin) commented Aug 19, 2026

Copy link
Copy Markdown
Member
  • Details
    • Remove various temporal comments that referred to earlier and/or later phases on the versioning work.
Microsoft Reviewers: Open in CodeFlow

Gudge and others added 2 commits August 18, 2026 16:56
This PR adds exhaustive state-aware adapters for the mutable 0.8.0-alpha
configuration contract.

Details

* Map every state-aware lifecycle and provision request into the current wire
  model.
* Preserve validated experimental data and source text in a neutral
  pre-normalization value.
* Add development-request facade coverage plus comprehensive expected-wire and
  current-wire equivalence tests.

Tests

* `cargo fmt --all -- --check`
* `cargo check -p wxc_common`
* `cargo clippy -p wxc_common --all-targets -- -D warnings`
* `cargo test -p wxc_common`

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 1ef43cc5-5752-426e-a36e-eb9aaccfb9ad
* Details
    - Remove various temporal comments that referred to earlier and/or later
    phases on the versioning work.
@MGudgin
Gudge (MGudgin) requested a review from a team as a code owner August 19, 2026 00:16
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@MGudgin
Gudge (MGudgin) force-pushed the user/gudge/version_specific_config_parsers_phase5d branch from adfc678 to 24b7408 Compare August 19, 2026 03:12
@MGudgin

Copy link
Copy Markdown
Member Author

Folded into #941: the marker destructuring and the non-temporal allow(dead_code) comments are now part of the Add 0.8 state-aware configuration adapters commit, so this branch no longer carries a distinct change.

@MGudgin
Gudge (MGudgin) deleted the user/gudge/version_specific_config_parsers_phase5d.1 branch August 19, 2026 14: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