Major: v3#320
Merged
Merged
Conversation
e33c22f to
9629f55
Compare
Added the `LibGit2Sharp` package (v0.31.0) to the `_atom` project for enhanced Git repository interaction capabilities.
…n labels Updated workflows to use `WorkflowLabels` for platform, framework, and version definitions for better consistency and readability. Removed unused classes and redefined multiple injection options. Migrated `GithubIfBuild` test cases and adjusted workflow test cases accordingly.
Added `DecSm.Atom.Workflows.Definition` to global usings for improved accessibility across the module.
Refactored `WithGithubRunsOnMatrix` and `WithGithubTokenInjection` methods for improved consistency and readability. Updated `GithubRunsOn` to use `WorkflowExpression` for labels. Removed redundant static properties for predefined runner configurations.
Added `EnvVarName` computed property to `ParamModel` and `ParamDefinition` for consistent environment variable naming. Updated `ParamService` to leverage the new property, improving readability and reducing duplication.
Enhanced workflow generation logic by introducing `IWorkflowExpressionGenerator` for improved expression handling and consistency. Refactored multiple areas, including runs-on labels, environment variables, and secret injections, to leverage the new generator. Simplified property access and reduced redundancy across GitHub and DevOps workflow writers.
…ndancy Removed redundant static properties and refactored workflow option records for improved clarity and consistency. Simplified the structure of options like `ToggleWorkflowOption`, `GithubCheckoutOption`, and `WorkflowSecretsInjection` by standardizing their definitions. Updated namespaces to align with the new organization.
…tions Removed the `IGithubExpression` record and its related implementations as they were no longer necessary. This includes literals, operators, function expressions, and tests under `ExpressionsTests`. Streamlined the codebase to improve maintainability and reduce unused types.
Simplified the `WithDevopsPoolMatrix` method implementation for improved clarity and consistency. Updated parameter handling and adjusted namespace dependencies to align with recent workflow architecture changes.
…ng change checks Introduced new workflow expression functionalities like `Not`, `And`, `Or`, `Coalesce`, and others to enhance GitHub workflow definition capabilities. Added logic for checking pull requests against potential breaking changes using the public API surface. Integrated `ICheckPrForBreakingChanges` and `IWorkflowExpressionWriter` for improved workflow management and customization.
Moved `CheckPrForBreakingChanges` logic in `Build.cs` to a specific workflow section and updated its injection configuration. Ensured consistency with GitHub token permissions and matrix setup. Removed redundant duplicate declarations for improved clarity.
Updated `GithubWorkflowWriter` to include `workflowStep.Options` in the processing of environment and parameter injections. Ensured that environment injections correctly exclude matching parameter injections to maintain expected behavior.
Updated `Validate.yml` and `Build.cs` to include `pull-request-number` parameter for PR breaking change checks. Ensured proper injection of GitHub event data to support enhanced workflow logic.
Refactored logic in `ICheckPrForBreakingChanges` to enhance clarity and functionality. Introduced separate handling for breaking change comments and non-breaking change comments. Prevented redundant mutation executions and improved error handling for mutation results.
Added `AtomFileSystem.cs` to the list of files checked for breaking changes in `ICheckPrForBreakingChanges`. Improved repository variable handling by splitting and formatting GitHub repository strings.
Improved handling of breaking change detection by introducing clearer formatting of repository variables and generating detailed draft PR review threads. Included file-specific line details for breaking changes to assist reviewers in identifying affected areas. Removed redundant code execution for review thread creation.
…flows Centralized breaking change detection logic into `SetupBuildInfo` and removed `ICheckPrForBreakingChanges` target to streamline workflow execution. Enhanced PR review comment generation with detailed breaking change insights. Updated workflows to ensure proper permissions and environment variable handling for seamless integration.
Enhanced workflows by injecting Azure Vault secrets and environment variables to support secure operations. Refactored breaking change detection in PR reviews for better clarity and streamlined mutation execution.
- Changed `AllowFailedResult` to `true` in the first test and `false` in the second one. - Ensures consistent behavior when handling failed test results.
- Updated `GithubTokenPermissionsOption` to use `BuildOptions.Github.TokenPermissions.Set`. - Aligns token permissions configuration with updated API conventions.
…ions - Added logic to detect and throw `StepFailedException` when `dotnet test` fails before producing a TRX file due to errors like build issues or invalid arguments. - Improved error handling to prevent misleading exit codes when no tests are run. - Updated test report generation to use the validated TRX file path.
…e operations - Introduced `GitVersionCache` to cache GitVersion outputs based on commit hash for improved performance. - Added `BuildCache` to skip redundant `dotnet build` operations by hashing build inputs. - Developed `RestoreCache` to avoid redundant `dotnet restore` using input-hash-based caching. - Created utility `HashCache` for common hashing logic leveraged by restore/build caches. - Integrated cache files into `obj` directories for natural cleanup via `dotnet clean`. - Enhanced safety with error handling and best-effort cleanup for locked or inaccessible files.
- Lowered `Microsoft.CodeAnalysis.CSharp` and `Microsoft.CodeAnalysis.Workspaces.Common` from 5.3.0 to 5.0.0 in library projects. - Adjusted `Microsoft.CodeAnalysis.CSharp.Workspaces` from 5.3.0 to 5.0.0 across test projects. - Ensures compatibility with dependent systems requiring an earlier version.
- Added `IBuildAccessor` inheritance to `IBuildDefinition` for expanded functionality. - Facilitates streamlined access to build-related operations via a unified interface.
- Expanded the examples section to include Hello World, adding params, secrets, and dependencies. - Provided step-by-step guides for project setup, using different features like parameters and secrets. - Included instructions for generating GitHub workflows with YAML examples. - Improved clarity and usability for new users exploring library features.
- Enabled `<Using>` directives in props files for all relevant modules, including `Dotnet`, `Build`, `Workflows`, `GitVersion`, `AzureStorage`, `AzureKeyVault`, and others. - Restores direct imports for global use.
- Removed unused `AzureKeyVault` imports, interfaces, and options from `_atom`. - Deleted project reference to `Invex.Atom.Module.AzureKeyVault` in `_atom.csproj`. - Cleaned up build definitions and options to exclude `AzureKeyVault` dependencies.
…ries - Clarified that `TransformFileScope`, `IRootedFileSystem`, and `IProcessRunner` are part of standalone packages (`Invex.FileSystem`, `Invex.Process`) maintained in separate repositories. - Updated package descriptions in documentation to reflect the separation of core foundational libraries from the main Atom packages. - Modified repository link in `docfx.json` to point to the correct `Invex-Games` namespace.
7f7e71e to
6dc5de4
Compare
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.
No description provided.