Refactor Hosts (again) and Reconcile Project & Solution referecnces. #2225
Conversation
This file contains 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
…t and update project references 🔧 (docs): update generated XML documentation to reflect latest commit and branch information The NKDAgility.AzureDevOps.Tools.CommandHost project is removed from the solution to streamline the project structure. This change also updates the project references to point to the correct paths, ensuring that the solution builds correctly. The generated XML documentation is updated to reflect the latest commit and branch information, ensuring that the documentation is accurate and up-to-date. ♻️ (refactor): update project references and remove unused code - Update project references to use `MigrationTools.Helpers.Tests` instead of `MigrationTools.Fakes`. - Rename `NKDAgility.AzureDevOps.Tools.CommandHost` project to `MigrationTools.Helpers.Tests`. - Add new fake classes `FakeMigrationToolVersion` and `FakeMigrationToolVersionInfo` for testing. - Replace `RunMigrationTools` with `RunConsoleAsync` in `Program.cs` files. - Remove unused `HostExtensions` and `MigrationService` classes. These changes improve the project structure by consolidating test helpers into a single project, ensuring consistency in project references, and removing obsolete code. This refactor enhances maintainability and clarity in the codebase. ♻️ (MigrationTools.Host): refactor to use Spectre.Console.Extensions.Hosting and remove redundant services 🔧 (MigrationTools.Host): update project references to reflect new structure Refactor the code to use `Spectre.Console.Extensions.Hosting` instead of `Spectre.Console.Cli.Extensions.DependencyInjection` for better integration and maintainability. Remove redundant services like `StartupService` and `DetectVersionService` to simplify the codebase. Update project references to reflect the new structure, ensuring that tests and other dependencies are correctly aligned with the changes. This improves the overall architecture and reduces complexity. ♻️ (tests): update namespace and project references from MigrationTools.Fakes to MigrationTools.Helpers.Tests The namespace and project references are updated to reflect the new location and naming convention for the test helpers. This change improves code organization and clarity by using a more descriptive and accurate namespace.
github-actions
bot
added
enhancement
documentation
…ndBase, ExecuteMigrationCommand, MigrationToolHost): refactor error handling and telemetry configuration - Update log messages to provide more context and clarity. - Remove telemetry tracking for specific exceptions to reduce noise. - Replace exceptions with critical log messages and exit the application for better control flow. - Add support for disabling telemetry via settings in CommandBase. - Refactor logging configuration to exclude specific Microsoft sources for cleaner logs. These changes improve error handling by providing more informative log messages and better control over application flow. Disabling telemetry based on user settings enhances privacy and control for users. The logging configuration changes help in maintaining cleaner and more relevant logs.
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.
♻️ (solution): remove NKDAgility.AzureDevOps.Tools.CommandHost project and update project references
🔧 (docs): update generated XML documentation to reflect latest commit and branch information
The NKDAgility.AzureDevOps.Tools.CommandHost project is removed from the solution to streamline the project structure. This change also updates the project references to point to the correct paths, ensuring that the solution builds correctly. The generated XML documentation is updated to reflect the latest commit and branch information, ensuring that the documentation is accurate and up-to-date.
♻️ (refactor): update project references and remove unused code
MigrationTools.Helpers.Testsinstead ofMigrationTools.Fakes.NKDAgility.AzureDevOps.Tools.CommandHostproject toMigrationTools.Helpers.Tests.FakeMigrationToolVersionandFakeMigrationToolVersionInfofor testing.RunMigrationToolswithRunConsoleAsyncinProgram.csfiles.HostExtensionsandMigrationServiceclasses.These changes improve the project structure by consolidating test helpers into a single project, ensuring consistency in project references, and removing obsolete code. This refactor enhances maintainability and clarity in the codebase.
♻️ (MigrationTools.Host): refactor to use Spectre.Console.Extensions.Hosting and remove redundant services
🔧 (MigrationTools.Host): update project references to reflect new structure
Refactor the code to use
Spectre.Console.Extensions.Hostinginstead ofSpectre.Console.Cli.Extensions.DependencyInjectionfor better integration and maintainability. Remove redundant services likeStartupServiceandDetectVersionServiceto simplify the codebase. Update project references to reflect the new structure, ensuring that tests and other dependencies are correctly aligned with the changes. This improves the overall architecture and reduces complexity.♻️ (tests): update namespace and project references from MigrationTools.Fakes to MigrationTools.Helpers.Tests
The namespace and project references are updated to reflect the new location and naming convention for the test helpers. This change improves code organization and clarity by using a more descriptive and accurate namespace.