Skip to content

refactor: optimized github actions workflow#112

Merged
OutdatedGuy merged 2 commits intomainfrom
workflow
Dec 16, 2025
Merged

refactor: optimized github actions workflow#112
OutdatedGuy merged 2 commits intomainfrom
workflow

Conversation

@OutdatedGuy
Copy link
Owner

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR optimizes the GitHub Actions workflows for a Flutter library by adding path filtering to reduce unnecessary workflow runs, implementing concurrency controls to cancel redundant jobs, and adding job timeouts for better resource management. The changes remove boilerplate comments and add push event triggers with the same path filters as pull requests.

Key changes:

  • Added path-based filtering to trigger workflows only on relevant file changes (Dart files, pubspec files, config files, and workflow files)
  • Implemented concurrency groups with cancel-in-progress to prevent redundant workflow runs
  • Added 5-minute timeouts to all jobs to prevent runaway processes

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
.github/workflows/unit_tests.yml Added path filters for pull_request and push events, concurrency control, and job timeout; removed boilerplate comments
.github/workflows/code_quality.yml Added path filters for pull_request and push events, concurrency control, and job timeouts for both lint and analyze jobs; removed boilerplate comments

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OutdatedGuy OutdatedGuy merged commit 1a08569 into main Dec 16, 2025
9 checks passed
@OutdatedGuy OutdatedGuy deleted the workflow branch December 16, 2025 19:13
@OutdatedGuy OutdatedGuy mentioned this pull request Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants