Skip to content

Conversation

@iamimmanuelraj
Copy link
Member

This pull request makes a small update to the GitHub Actions workflow by removing the step that installs system dependencies such as zip, unzip, and composer from the .github/workflows/release_on_tag.yml file. This change simplifies the workflow and may reduce build times.

Already installed in runner

Signed-off-by: Immanuel Raj <iamimmanuelraj@gmail.com>
Signed-off-by: Immanuel Raj <iamimmanuelraj@gmail.com>
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 pull request simplifies GitHub Actions workflows by removing manual system dependency installation steps and modernizing the release artifact upload process. The changes aim to reduce build times and leverage more reliable GitHub CLI tooling.

  • Removes manual installation of system dependencies (zip, unzip, composer, git, sudo) from workflows
  • Replaces the softprops/action-gh-release action with GitHub CLI commands for release management
  • Streamlines workflow execution by relying on pre-installed dependencies in GitHub runners

Reviewed Changes

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

File Description
.github/workflows/release_on_tag.yml Removes system dependency installation and replaces release action with gh CLI commands
.github/workflows/phpcs_on_pull_request.yml Removes unnecessary git and sudo installation step

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mrrobot47 mrrobot47 merged commit 3c8b19d into main Aug 14, 2025
5 checks passed
@mrrobot47 mrrobot47 deleted the actions-fix branch August 14, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants