-
Notifications
You must be signed in to change notification settings - Fork 32
Update ModularPipelines to v3 (major) #39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the ModularPipelines NuGet packages from version 2.48.30 to 3.0.1, which is a major version upgrade. However, the PR only includes package version updates without the necessary code changes to support the v3 API.
Changes:
- Updated ModularPipelines package from 2.48.30 to 3.0.1
- Updated ModularPipelines.Git package from 2.48.30 to 3.0.1
- Updated ModularPipelines.GitHub package from 2.48.30 to 3.0.1
- Updated ModularPipelines.DotNet package from 2.48.30 to 3.0.1
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
build/Build.csproj
Outdated
| <PackageReference Include="ModularPipelines" Version="3.0.1"/> | ||
| <PackageReference Include="ModularPipelines.Git" Version="3.0.1"/> | ||
| <PackageReference Include="ModularPipelines.GitHub" Version="3.0.1"/> | ||
| <PackageReference Include="ModularPipelines.DotNet" Version="3.0.1"/> |
Copilot
AI
Jan 17, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This package update from ModularPipelines v2.48.30 to v3.0.1 is a major version upgrade with significant breaking changes, but the PR does not include the required code changes to make the build system compatible with v3. The current codebase uses the v2 API extensively and will not compile with v3 packages.
According to the v3 release notes, the following code changes are required but missing from this PR:
- Program.cs needs to replace PipelineHostBuilder.Create() with Pipeline.CreateBuilder(args), and change from callback-based configuration to direct property access
- All modules need to change IPipelineContext to IModuleContext in ExecuteAsync signatures
- GetModule calls need to be moved from module instances to the context parameter
- Module result access needs to be updated from .Value/.HasValue to pattern matching or .ValueOrDefault
- Module configuration properties (ModuleRunType, ShouldIgnoreFailures, OnAfterExecute) need migration to the new Configure() fluent API or lifecycle methods
Without these changes, the build will fail to compile. This PR should either include all necessary code migrations or be split into smaller PRs that update both packages and code together.
| <PackageReference Include="ModularPipelines" Version="3.0.1"/> | |
| <PackageReference Include="ModularPipelines.Git" Version="3.0.1"/> | |
| <PackageReference Include="ModularPipelines.GitHub" Version="3.0.1"/> | |
| <PackageReference Include="ModularPipelines.DotNet" Version="3.0.1"/> | |
| <PackageReference Include="ModularPipelines" Version="2.48.30"/> | |
| <PackageReference Include="ModularPipelines.Git" Version="2.48.30"/> | |
| <PackageReference Include="ModularPipelines.GitHub" Version="2.48.30"/> | |
| <PackageReference Include="ModularPipelines.DotNet" Version="2.48.30"/> |
d68bacb to
ea4bb08
Compare
This PR contains the following updates:
2.48.30→3.0.862.48.30→3.0.862.48.30→3.0.862.48.30→3.0.86Release Notes
thomhurst/ModularPipelines (ModularPipelines)
v3.0.86: 3.0.86What's Changed
Full Changelog: thomhurst/ModularPipelines@v3.0.84...v3.0.86
v3.0.1: 3.0.1What's Changed
Full Changelog: thomhurst/ModularPipelines@v3.0.0...v3.0.1
v3.0.0: 3.0.0ModularPipelines V3 Release Notes
Highlights
ASP.NET Core-Style Builder Pattern
No more callbacks. Direct property access, just like ASP.NET Core minimal APIs.
If you've used ASP.NET Core, this feels instantly familiar.
Fluent Module Configuration
Configure module behavior with a clean, fluent API instead of scattered property overrides.
Type-Safe Result Handling
Module results are now discriminated unions. Pattern matching gives you compile-time safety.
Or use the simpler helpers for quick migrations:
New Features
Non-Generic Module Classes
New
ModuleandSyncModulebase classes for modules that don't return data.Internally these use the
Nonestruct, which represents "nothing" and is semantically equivalent tonull.Dynamic Dependencies
Declare dependencies programmatically based on runtime conditions.
Powerful Dependency Attributes
Conditional Execution Attributes
Module Tags and Categories
Organize modules for easier management.
Pipeline Validation
Catch configuration errors before execution.
Plugin System
Create reusable pipeline extensions.
Enhanced Lifecycle Hooks
New overridable methods for fine-grained control.
Breaking Changes
Entry Point
PipelineHostBuilder.Create()Pipeline.CreateBuilder(args).ConfigureAppConfiguration(callback)builder.Configuration.ConfigureServices(callback)builder.Services.ConfigurePipelineOptions(callback)builder.Options.AddModule<T>()on builderbuilder.Services.AddModule<T>().ExecutePipelineAsync().Build().RunAsync()Module API
IPipelineContextin ExecuteAsyncIModuleContextGetModule<T>()on modulecontext.GetModule<T>()Timeoutproperty overrideConfigure().WithTimeout()RetryPolicyproperty overrideConfigure().WithRetryCount()ShouldSkip()methodConfigure().WithSkipWhen()ShouldIgnoreFailures()methodConfigure().WithIgnoreFailures()ModuleRunType.AlwaysRunConfigure().WithAlwaysRun()OnBeforeExecute()Configure().WithBeforeExecute()orOnBeforeExecuteAsync()OnAfterExecute()Configure().WithAfterExecute()orOnAfterExecuteAsync()Result Access
result.Valueresult.ValueOrDefaultor pattern matchresult.Exceptionresult.ExceptionOrDefaultor pattern matchresult.ModuleResultType == ModuleResultType.Successresult.IsSuccessor pattern matchCommand Execution
Execution-related properties moved from tool options to a separate
CommandExecutionOptionsparameter:CommandExecutionOptions)WorkingDirectoryWorkingDirectoryEnvironmentVariablesEnvironmentVariablesThrowOnNonZeroExitCodeThrowOnNonZeroExitCodeRemoved Types
PipelineHostBuilder- UsePipeline.CreateBuilder()ModuleBase/ModuleBase<T>- UseModule<T>Migration Path
Quick Migration (Minimal Changes)
The
ExecutePipelineAsync()extension still exists:And
ValueOrDefaultprovides backwards-compatible result access:Full Migration
For the cleanest code, adopt the new patterns:
Pipeline.CreateBuilder(args)with direct property accessConfigure()builderIPipelineContexttoIModuleContextGetModule<T>()calls to contextSee the Migration Guide for detailed examples.
Upgrade Steps
dotnet add package ModularPipelines --version 3.0.0Getting Help
migrationlabelWhat's Changed
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR has been generated by Renovate Bot.