Enhance Build Error Analysis with Improved Artifact Handling and PR Triggers#5
Merged
Merged
Conversation
…riggers This PR adds significant improvements to the build error analysis system: 1. **Improved Artifact Management**: - Added a dedicated `artifact-manager.sh` script that searches for build logs in artifacts - Implements intelligent log file detection that can identify build logs by content - Combines multiple logs into a single file for comprehensive analysis 2. **Dual-Trigger Workflow**: - Added direct `pull_request` trigger alongside the existing `workflow_run` trigger - PR trigger runs the build and analyzes errors directly from the PR context - Workflow run trigger continues to analyze artifacts from completed workflows 3. **Better Error Detection**: - Improved pattern matching for various log file formats - Added fallback mechanisms when primary log files aren't found - More reliable artifact extraction and processing 4. **Enhanced Reporting**: - Maintains a manifest of all discovered log files - Separates logs into individual files for targeted analysis - Preserves the combined log for comprehensive error detection These changes make the error analysis system much more robust, ensuring it works properly with both PR workflows and standalone builds.
Author
Enhanced Build Error AnalysisThis PR implements a complete overhaul of the build error analysis system with two key improvements: 1. Dual-Trigger WorkflowThe workflow now has two separate trigger mechanisms: Pull Request Trigger (Direct Analysis)pull_request:
types: [opened, synchronize, reopened]
Workflow Run Trigger (Artifact Analysis)workflow_run:
workflows: ["Create New Release"]
types: [completed]
2. Improved Artifact HandlingThe new
3. Enhanced Error DetectionThe system now uses a more advanced approach to find and process errors:
These changes make the error analysis system much more reliable across different contexts and ensure it works properly regardless of whether it's triggered by PRs or workflow runs. |
Automatically fixed code quality issues using SwiftLint, SwiftFormat, and Clang-Format.
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.
This PR adds significant improvements to the build error analysis system:
Improved Artifact Management:
artifact-manager.shscript that searches for build logs in artifactsDual-Trigger Workflow:
pull_requesttrigger alongside the existingworkflow_runtriggerBetter Error Detection:
Enhanced Reporting:
These changes make the error analysis system much more robust, ensuring it works properly with both PR workflows and standalone builds.
🤖 See my steps and cost here ✨
#1