Update Artifact Handling and Add PR Trigger to Error Analysis Workflow#6
Merged
Merged
Conversation
This PR updates the existing error analysis system with improved artifact handling and adds a direct PR trigger: 1. **Enhanced Artifact Handling**: - Updated `download-artifacts.sh` with better log file detection - Added multiple fallback mechanisms for finding build logs - Improved handling of multiple artifact formats - Added automatic log file combination for comprehensive analysis 2. **Direct PR Trigger**: - Added pull_request trigger alongside the existing workflow_run trigger - Added a dedicated job for analyzing PRs directly - Builds the project and analyzes errors in a single workflow 3. **Better Maintainability**: - Moved more inline code from the workflow file to scripts - Improved error handling and log output - Added better fallback mechanisms when resources are missing These changes make the error analysis system more reliable by ensuring it finds and analyzes logs in more situations, and make the system easier to maintain by moving complex logic out of the workflow file into dedicated scripts.
Author
Enhanced Artifact Handling and PR TriggerThis PR updates the existing build error analysis system to: 1. Improve Artifact HandlingI've enhanced the existing
2. Add Direct PR TriggerAdded
3. Move Code to ScriptsCreated dedicated scripts to clean up the workflow file:
These changes ensure the error analysis system is more reliable at finding and analyzing logs, while making the workflow file much cleaner and more maintainable. |
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 updates the existing error analysis system with improved artifact handling and adds a direct PR trigger:
Enhanced Artifact Handling:
download-artifacts.shwith better log file detectionDirect PR Trigger:
Better Maintainability:
These changes make the error analysis system more reliable by ensuring it finds and analyzes logs in more situations, and make the system easier to maintain by moving complex logic out of the workflow file into dedicated scripts.
🤖 See my steps and cost here ✨
#1