AutoVersioningEnabled#3
Merged
Merged
Conversation
Enhanced the CI and release process in `dotnet.yml` by updating the `push` event to include all branches, ensuring a full repository clone for accurate versioning with Nerdbank.GitVersioning. The test step has been renamed for clarity, and the version output variable has been updated to `SemVer2`. Additionally, the release job now features a more detailed setup for creating GitHub releases, including proper tagging and automatic release notes generation.
This commit introduces the `Nerdbank.GitVersioning` package for improved version management in the project. Several files, including `ChatBot.razor`, `AIServiceFactory.cs`, and `CompanyService.cs`, were refactored to consistently use the `this` keyword for member variables, enhancing code readability and maintaining a consistent coding style. Additionally, the `ConfigurationService.cs` was updated to improve performance by changing return types to empty arrays, and centralized package version management was implemented through the new `Directory.Packages.props` file.
This commit introduces a new step in the GitHub Actions workflow to automatically locate the project file for publishing. The "Detect publishable project" step uses PowerShell to prioritize web projects and falls back to the first `.csproj` file if none are found. The detected project path is then utilized in the publish step, replacing the previous hardcoded project file reference.
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
1 participant
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.
PR Classification
Enhancement of CI/CD pipeline and code cleanup.
PR Summary
This pull request updates the CI/CD pipeline to support all branches and improves code readability and maintainability across several files. Key changes include:
dotnet.yml: Enhanced CI/CD pipeline with support for all branches and added release job.ChatBot.razor: Method name changed toSendMessageAsyncfor asynchronous operation.AIServiceFactory.cs,ChatBotService.cs,CompanyService.cs,ProblemReportService.cs,ServiceNowDataGeneratorService.cs,ServiceNowDataImporter.cs,TrendingService.cs: Addedthisfor member variable references to improve clarity.AIForITSM.csproj: Updated package references for flexibility and addedNerdbank.GitVersioning.Directory.Build.propsandDirectory.Packages.props: Introduced to centralize project settings and manage package versions.