chore: TimeWarp standards, TW#### diagnostics, modern CI#31
Merged
Conversation
Architecture now owns TWA####; this package must move TWA001/TWA002 (and clean TW/TWG stew) to a distinct prefix (recommend TWS).
Clarify: rename every diagnostic in this package to one prefix (TWS); do not leave TWA001/TWA002. Product hygiene, not optional collision fix.
Recommend TW0001–TW0006; Architecture keeps TWA. Drop TWS suggestion.
Bring the repo to pass `ganda repo audit` by scaffolding the dev CLI, standard MSBuild/CPM layout, banned-API rules, memsearch hooks, and VS Code window branding, and by targeting net10.0 so Nuru/Amuru package assets resolve.
Track thin YAML + OIDC + dev-cli pipeline work after the standards audit left the old PowerShell release flow in place.
Move task to in-progress for orchestration.
Capture inventory, ordered steps, verification, and out-of-scope after planning.
Drop TWA/TWG/TW1xxx so TimeWarp Architecture can own TWA. Single TW family for TimeWarp.SourceGenerators analyzers and generators; update Unshipped releases, docs, editorconfig, and open kanban references.
Match live interface-delegation generator wording after 018 rename review.
Document mapping, files, verification, and review outcomes before done.
Move task to done after rename implementation and review.
Move task to in-progress for orchestration.
Capture peer template, T1 ProjectReference, workflow-command rewrite, and path/cache/auth decisions.
Replace hand-rolled PowerShell CI with peer-style OIDC workflow and in-process dev workflow (clean/build/test/push). Wire test console as Analyzer ProjectReference so CI verifies generators without local-feed version chicken-and-egg.
Document YAML, workflow-command, T1 ProjectReference, verification, and ops notes.
Move task to done after Option A CI implementation and review.
NuGet fails with NU1301 when nuget.config points at artifacts/packages and the directory is missing (CI checkout). Peers use nuget.org only; PackageOutputPath still writes nupkgs for upload/push.
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
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.
Summary
Bring this repo onto current TimeWarp standards and free TWA for Architecture.
tools/dev-cli, banned APIs, memsearch hooks,workflow.ymlrename, net10.0 alignment, VS Code window brandingdev workflow(OIDC Trusted Publishing on release; clean → build → test; release push via GeneratePackageOnBuild)Package version remains 1.0.0-beta.8 (greater than latest release v1.0.0-beta.7).
Diagnostic mapping
Breaking (beta): consumers must update
dotnet_diagnostic.*keys / suppressions from old IDs to TW0001–TW0006.CI
tools/**,Directory.Packages.props,msbuild/**nuget/login@v1only onrelease(noPUBLISH_TO_NUGET_ORG)ProjectReferenceso CI exercises in-tree generatorsTest plan
dotnet build source/timewarp-source-generators/... -c Releasesucceedsdotnet run --file tools/dev-cli/dev.cs -- workflow(PR mode) exit 0Ops notes
PUBLISH_TO_NUGET_ORGwhen convenient