Skip to content

Commit 5d3160d

Browse files
thomhurstclaude
andcommitted
fix: Keep IParallelLimitProvider and GetLock internal
Address PR review feedback - the visibility change from internal to public was accidental. IParallelLimitProvider is internal infrastructure and should not be part of the public API surface. - Revert IParallelLimitProvider from public to internal - Change ParallelLimiterAttribute.GetLock to internal 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent ae34fe5 commit 5d3160d

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/ModularPipelines/Attributes/ParallelLimiterAttribute.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public ParallelLimiterAttribute() : base(typeof(TParallelLimit))
1717
}
1818

1919
/// <inheritdoc />
20-
public override AsyncSemaphore GetLock(IParallelLimitProvider provider)
20+
internal override AsyncSemaphore GetLock(IParallelLimitProvider provider)
2121
{
2222
return provider.GetLock<TParallelLimit>();
2323
}
@@ -52,5 +52,5 @@ protected ParallelLimiterAttribute(Type type)
5252
/// </summary>
5353
/// <param name="provider">The parallel limit provider.</param>
5454
/// <returns>The semaphore for this limit type.</returns>
55-
public abstract AsyncSemaphore GetLock(IParallelLimitProvider provider);
55+
internal abstract AsyncSemaphore GetLock(IParallelLimitProvider provider);
5656
}

src/ModularPipelines/Helpers/IParallelLimitProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
namespace ModularPipelines.Helpers;
66

7-
public interface IParallelLimitProvider
7+
internal interface IParallelLimitProvider
88
{
99
/// <summary>
1010
/// Gets a semaphore lock for the specified parallel limit type.

0 commit comments

Comments
 (0)