-
Notifications
You must be signed in to change notification settings - Fork 383
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
workflow CodeQL 运行失败 #307
Comments
@Kyle-Ye 有空可以帮忙看看这个问题吗 |
2023-12-31T02:28:35.9085640Z /Users/runner/work/Easydict/Easydict/Easydict/NewApp/View/MenuItemView.swift:92:1: error: expected expression
2023-12-31T02:28:35.9087060Z @available(macOS 13, *)
2023-12-31T02:28:35.9087490Z ^
2023-12-31T02:28:35.9088790Z /Users/runner/work/Easydict/Easydict/Easydict/NewApp/View/SettingView/Tabs/GeneralTab.swift:53:1: error: expected expression
2023-12-31T02:28:35.9090350Z @available(macOS 13, *)
2023-12-31T02:28:35.9090770Z ^
2023-12-31T02:28:35.9092030Z /Users/runner/work/Easydict/Easydict/Easydict/NewApp/View/SettingView/Tabs/AboutTab.swift:59:1: error: expected expression
2023-12-31T02:28:35.9093540Z @available(macOS 13, *)
2023-12-31T02:28:35.9093960Z ^
2023-12-31T02:28:35.9095680Z /Users/runner/work/Easydict/Easydict/Easydict/NewApp/View/MenuItemView.swift:66:13: error: cannot find 'SettingsLink' in scope
2023-12-31T02:28:35.9097250Z SettingsLink()
2023-12-31T02:28:35.9097690Z ^~~~~~~~~~~~
2023-12-31T02:28:35.9097980Z I believe this is because the CI is using And after #212, the minimal supported version to build the project is Xcode 15.0. We can probably solve the issue by upgrading the CI's Xcode version.
|
目前项目使用的是 GitHub 仓库设置中的 CodeQL,我从 action 市场这里的配置文件看到已经是 macos-latest,也就是说是 macOS 13,那这里还能改吗?
|
You may try to specify Xcode version to 15.0 and give it a try. Also I noticing you are adding This will not work since the EasyDict project is a xcodeproj not a swift package. For a Swift package, both
https://github.com/Kyle-Ye/MenuHelper
eg.
|
Every macOS runner image have a default Xcode version. And it will bundle a lot of different Xcode version. See more info here https://github.com/actions/runner-images/blob/main/images/macos/macos-13-Readme.md In our case, we need to manually update the default Xcode version. You can do it by hand or using a Xcode action.(eg. https://github.com/maxim-lobanov/setup-xcode) |
ok,我先来试一下。 |
Most of my macOS project's CI is using Xcode Cloud. See I can't find a reference workflow file for you. Here is a iOS one in case you need something for reference. |
CodeQL 已经修复了,通过指定 Runner Image 为 macos-13 和设置 xcode-version: '15.1' https://github.com/tisfeng/Easydict/blob/main/.github/workflows/codeql.yml 之前没注意 macos-latest 居然是 macos-12,这太坑了 🥲
|
这个 workflow 之前运行很好,我一直没动它,最近突然运行失败了 https://github.com/tisfeng/Easydict/actions/workflows/github-code-scanning/codeql
我看了一下,是从这个提交开始失败 https://github.com/tisfeng/Easydict/actions/runs/7367547129
中间间隔两个提交,是我将 AppCenter 和 firebase 从 Cocoapod 改成了 SPM,本地编译是没问题的,不知道为什么 CI 报错,我对 Swift 不太了解,不知道哪里出了问题。
The text was updated successfully, but these errors were encountered: