Skip to content

Fix CI workflow issues flagged in Copilot review of PR #12#14

Merged
kiyarose merged 11 commits into
mainfrom
copilot/fix-copilot-review-issues
Mar 28, 2026
Merged

Fix CI workflow issues flagged in Copilot review of PR #12#14
kiyarose merged 11 commits into
mainfrom
copilot/fix-copilot-review-issues

Conversation

Copilot AI commented Mar 28, 2026

Copy link
Copy Markdown
Contributor
  • Fix xcodebuild archive -target incompatibility — switched to xcodebuild build … SYMROOT=build
  • Repackage iOS .app as a proper IPA (mkdir build/ipa/Payload, cp -r Flean.app, zip ../Flean.ipa Payload) in prerelease.yml
  • Update iOS upload step in prerelease.ymlFlean.ipa / application/octet-stream
  • Same IPA packaging change in build-and-release.yml
  • Update iOS upload step in build-and-release.ymlFlean.ipa / application/octet-stream

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

@deepsource-io

deepsource-io Bot commented Mar 28, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in c3467e8...b5f63e0 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
PHP Mar 28, 2026 2:44p.m. Review ↗
Shell Mar 28, 2026 2:44p.m. Review ↗
JavaScript Mar 28, 2026 2:44p.m. Review ↗
Swift Mar 28, 2026 2:44p.m. Review ↗

@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 14:05
Copilot AI review requested due to automatic review settings March 28, 2026 14:05
@kiyarose kiyarose marked this pull request as draft March 28, 2026 14:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the CI workflows introduced in PR #12 to be more reliable on GitHub-hosted runners (macOS + Ubuntu), primarily around version/tag detection and Xcode build invocations given that schemes aren’t committed.

Changes:

  • Fix version extraction and tag-existence checks (avoid shallow clone/tag-miss; avoid grep -P on macOS).
  • Add explicit GITHUB_TOKEN permissions and adjust xcodebuild usage to -project/-target + correct iOS destination.
  • Add iOS archive + exportArchive steps and correct artifact paths; fix macOS zip output location.

Reviewed changes

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

File Description
.github/workflows/build-and-release.yml Release pipeline: detect version/tag, build iOS + macOS, tag, create GitHub release, upload artifacts.
.github/workflows/prerelease.yml PR-ready prerelease pipeline: build iOS + macOS and create a prerelease tag/release.

Comment thread .github/workflows/build-and-release.yml Outdated
Comment thread .github/workflows/build-and-release.yml Outdated
Comment thread .github/workflows/prerelease.yml Outdated
Comment thread .github/workflows/prerelease.yml
Comment thread .github/workflows/build-and-release.yml Outdated
Comment thread .github/workflows/build-and-release.yml Outdated
Comment thread .github/workflows/build-and-release.yml Outdated
Copilot AI requested a review from kiyarose March 28, 2026 14:10
kiyarose and others added 3 commits March 28, 2026 10:11
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 14:12
@kiyarose kiyarose marked this pull request as draft March 28, 2026 14:15
@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 14:22
@kiyarose kiyarose marked this pull request as draft March 28, 2026 14:25
deepsource-autofix Bot and others added 3 commits March 28, 2026 14:28
This commit fixes the style issues introduced in 0a0cf7f according to the output
from PHP CS Fixer, StandardJS and swift-format.

Details: #14
@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 14:34
@kiyarose kiyarose marked this pull request as draft March 28, 2026 14:35
@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 14:38
@kiyarose kiyarose marked this pull request as draft March 28, 2026 14:43
@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 14:46
@kiyarose kiyarose marked this pull request as draft March 28, 2026 15:45
@kiyarose kiyarose marked this pull request as ready for review March 28, 2026 16:16

@kiyarose kiyarose left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good for now, any issues and im assassinating myself.

@kiyarose kiyarose merged commit 8df0a29 into main Mar 28, 2026
8 checks passed
@kiyarose kiyarose deleted the copilot/fix-copilot-review-issues branch March 28, 2026 16:17
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