Skip to content

Refactor DependencyCollisionDetector.cs to work on pipeline start, no…#4

Merged
thomhurst merged 3 commits intomainfrom
feature/dependency-collision
Jun 30, 2023
Merged

Refactor DependencyCollisionDetector.cs to work on pipeline start, no…#4
thomhurst merged 3 commits intomainfrom
feature/dependency-collision

Conversation

@thomhurst
Copy link
Owner

…t on module start

@thomhurst thomhurst temporarily deployed to Pull Requests June 30, 2023 15:34 — with GitHub Actions Inactive
@thomhurst thomhurst temporarily deployed to Pull Requests June 30, 2023 15:35 — with GitHub Actions Inactive
@thomhurst thomhurst temporarily deployed to Pull Requests June 30, 2023 16:16 — with GitHub Actions Inactive
@thomhurst thomhurst merged commit 2ad56b7 into main Jun 30, 2023
@thomhurst thomhurst deleted the feature/dependency-collision branch August 17, 2023 09:10
@claude claude bot mentioned this pull request Feb 22, 2026
5 tasks
thomhurst added a commit that referenced this pull request Feb 24, 2026
…cate shared code, pin cloudflared

- Add volatile to DeferredCoordinator/DeferredArtifactStore _inner fields
  for correct double-checked locking memory visibility (#9)
- Replace O(n) FirstOrDefault module lookups with Dictionary<string, IModule>
  built once per execution in both executors (#4)
- Extract DependencyResultApplicator with shared ApplyDependencyResults and
  PublishResolutionFailureAsync, eliminating duplication between
  DistributedModuleExecutor and WorkerModuleExecutor (#5)
- Pin cloudflared to version 2026.2.0 in CI workflow instead of using
  latest release (#8)
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