delete: remove Beutl.Operators project and related references#1612
Open
delete: remove Beutl.Operators project and related references#1612
Conversation
Contributor
|
No TODO comments were found. |
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the Beutl.Operators project from the solution and eliminates build/runtime references to it, aligning the codebase with the stated breaking change (Operators functionality removed).
Changes:
- Deleted
src/Beutl.Operatorsproject and removed it from the solution. - Removed project references to
Beutl.Operatorsfrom dependent projects and shared build props. - Updated core-library/dependency enumeration and app startup imports to no longer reference
Beutl.Operators.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/Beutl.UnitTests/Beutl.UnitTests.csproj | Drops the Beutl.Operators project reference from unit tests. |
| src/Beutl/Services/LibraryRegistrar.cs | Moves LibraryRegistrar into Beutl.Services namespace and removes Operators-related usings. |
| src/Beutl/App.axaml.cs | Removes using Beutl.Operators; from app startup code. |
| src/Beutl.Operators/Beutl.Operators.csproj | Removes the Operators project definition. |
| src/Beutl.Engine/Properties/AssemblyInfo.cs | Removes InternalsVisibleTo("Beutl.Operators"). |
| src/Beutl.Editor.Components/Beutl.Editor.Components.csproj | Drops the Beutl.Operators project reference. |
| src/Beutl.Api/Services/CoreLibraries.cs | Removes Beutl.Operators from the runtime dependency allowlist/switch. |
| nukebuild/Build.cs | Removes Beutl.Operators from build core-library lists. |
| build/props/CoreLibraries.props | Removes shared ProjectReference to Beutl.Operators. |
| Beutl.slnx | Removes Beutl.Operators project entry from the solution. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Contributor
Minimum allowed line rate is |
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.
Description
Remove the Beutl.Operators project and all related references from the solution.
Breaking changes
This change removes the Beutl.Operators functionality, which may affect any dependent projects.
Fixed issues
None