Skip to content

Commit 0c278f7

Browse files
committed
Remove archived benchmark clone references
1 parent 5a1cb63 commit 0c278f7

5 files changed

Lines changed: 1 addition & 6 deletions

File tree

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ node_modules/
2222

2323
# Benchmarks
2424
BenchmarkDotNet.Artifacts/
25-
benchmarks/CodecMapper/
2625
.fsdocs/
2726

2827
# IDEs

AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,5 +54,5 @@ These standards represent the user's preferred style and architectural philosoph
5454
- **Numeric parsing should stay on the shared portable helpers.** Route JSON/XML/KeyValue/import numeric decoding through the `Core.tryParse...Invariant` and `Core.parse...Invariant` helpers instead of ad hoc `Parse(..., InvariantCulture)` calls plus exception-type checks, so Fable stays warning-free and the invalid/out-of-range behavior remains aligned across runtimes.
5555
- **The C# facade is intentionally narrower than the bridge.** `CSharpSchema.Record(...)` is for new setter-bound C# classes and wraps the existing schema model; constructor-bound or attribute-driven C# contracts should still prefer the bridge or future codegen instead of stretching the facade into a second schema system.
5656
- **Do not use `System.Enum.ToObject` or `System.Convert.ChangeType` in the core portable path.** Fable rejects both APIs. When adding enum support, keep the .NET path behind `#if !FABLE_COMPILER` and use a Fable-safe erased-number path instead.
57-
- **BenchmarkDotNet now runs via the in-process emit toolchain.** This avoids child-project generation conflicts with the archived experimental clone under `benchmarks/CodecMapper/`, which also contains a `CodecMapper.Benchmarks.fsproj`. Keep the manual runner for quick snapshots and README numbers.
57+
- **BenchmarkDotNet now runs via the in-process emit toolchain.** Keep the manual runner for quick snapshots and README numbers.
5858
- **Project layout is now split by role.** Public libraries live under `src/`, executable and xUnit tests live under `tests/`, and benchmark apps live under `benchmarks/`. Keep new projects in the root that matches their purpose so tooling and docs discovery stay predictable.

notes/AGENT_NOTES.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ This keeps compilation cost visible and avoids hidden recompilation or implicit
147147
## Benchmarking Notes
148148

149149
- `benchmarks/CodecMapper.Benchmarks/CodecMapper.Benchmarks.fsproj` is now a runnable BenchmarkDotNet app again (`OutputType=Exe` plus a real entrypoint), so the failure is no longer at the top-level `dotnet run` step.
150-
- The archived experimental clone under `benchmarks/CodecMapper/` also contains a `CodecMapper.Benchmarks.fsproj`, which makes BenchmarkDotNet's default child-project generator ambiguous once both repos exist in the same workspace.
151150
- `benchmarks/CodecMapper.Benchmarks/Program.fs` now forces BenchmarkDotNet onto `InProcessEmitToolchain` to avoid child-project generation entirely.
152151
- The remaining warning during local runs is just Linux process-priority elevation failure (`Permission denied`), which does not stop benchmarks from executing.
153152
- A manual Release runner was added in `benchmarks/CodecMapper.Benchmarks.Runner` to keep benchmark reporting moving while that tooling issue remains unresolved.
@@ -187,7 +186,6 @@ This keeps compilation cost visible and avoids hidden recompilation or implicit
187186

188187
## Legacy CodecMapper Comparison
189188

190-
- The previous `CodecMapper` repo is cloned locally at `benchmarks/CodecMapper/` for reference only and is ignored by this repo's Git metadata.
191189
- Its published benchmark snapshot is not directly comparable to `CodecMapper`'s current README numbers because it benchmarks a 1000-record `Person list` payload, while `CodecMapper` currently publishes a small single-object benchmark.
192190
- The old repo also fails to complete BenchmarkDotNet runs cleanly on this machine under `.NET SDK 10.0.103`; direct child-project builds still end with `Build FAILED` and `0 Error(s)`.
193191
- Legacy branding assets from the archived experimental repo are preserved under `notes/legacy-codemapper/`.

scripts/format-check.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ cd "$repo_root"
88

99
mapfile -t files < <(
1010
find src tests benchmarks \
11-
-path 'benchmarks/CodecMapper' -prune -o \
1211
-path '*/bin' -prune -o \
1312
-path '*/obj' -prune -o \
1413
-type f \( -name '*.fs' -o -name '*.fsi' -o -name '*.fsx' \) \

scripts/format.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ cd "$repo_root"
88

99
mapfile -t files < <(
1010
find src tests benchmarks \
11-
-path 'benchmarks/CodecMapper' -prune -o \
1211
-path '*/bin' -prune -o \
1312
-path '*/obj' -prune -o \
1413
-type f \( -name '*.fs' -o -name '*.fsi' -o -name '*.fsx' \) \

0 commit comments

Comments
 (0)