Refactor workflows and update extension branding #10
Conversation
Consolidates jobs into a single build-and-test job, improves caching strategy, simplifies test result publishing, and enhances test summary output. The workflow now builds both Debug and Release configurations, provides clearer logs, and outputs test summaries directly to the GitHub Actions summary. Removes redundant code-quality and status-check jobs for a more streamlined process.
Replaces the old ExtractInterface.png with new InterfaceExtractor-logo.png and InterfaceExtractor-128x128.png. Updates the VSIX manifest to reference the new logo as the extension icon and includes it in the project file for packaging.
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the CI/CD workflow and updates the extension's branding for Visual Studio Marketplace publication by consolidating the test workflow and adding logo assets.
- Consolidated the PR test workflow from multiple jobs (test, code-quality, status-check) into a single build-and-test job
- Added new logo files and updated manifest references for marketplace branding
- Enhanced workflow with improved caching, detailed test reporting, and GitHub Actions summary output
Reviewed Changes
Copilot reviewed 3 out of 6 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/pr-tests.yml |
Consolidated workflow jobs, improved caching strategy, added test summary output to GitHub Actions |
InterfaceExtractor.Extension.csproj |
Added new logo resource with VSIX inclusion configuration |
InterfaceExtractor.Extension/source.extension.vsixmanifest |
Added icon reference for marketplace branding |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| with: | ||
| fetch-depth: 0 # Needed for better caching |
There was a problem hiding this comment.
The comment states fetch-depth: 0 is 'Needed for better caching', but fetch-depth doesn't affect caching. This parameter retrieves full git history and is typically used for changelog generation or git-based versioning. Consider clarifying the actual purpose or removing fetch-depth if not needed.
| with: | |
| fetch-depth: 0 # Needed for better caching |
| /ResultsDirectory:TestResults ` | ||
| /Parallel |
There was a problem hiding this comment.
[nitpick] The /Parallel flag may cause issues with test isolation or resource contention in CI environments. Consider testing without this flag first, or document why parallel execution is safe for these tests.
| /ResultsDirectory:TestResults ` | |
| /Parallel | |
| /ResultsDirectory:TestResults |
Added a check for commit messages starting with 'patch' to trigger a PATCH version bump in the release workflow. This ensures that patch-level changes are properly recognized and versioned.
Description
This PR improves the CI/CD workflow and updates the extension's branding for Visual Studio Marketplace publication.
Changes
Workflow improvements:
Branding updates:
Resources/folderFiles Changed
.github/workflows/pr-test.ymlInterfaceExtractor.Extension.csprojsource.extension.vsixmanifestResources/InterfaceExtractor-logo.png(new)Resources/InterfaceExtractor-128x128.png(new)Testing