Skip to content
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

fix sed in ci runs on macos #662

Merged
merged 1 commit into from
Jan 25, 2025
Merged

fix sed in ci runs on macos #662

merged 1 commit into from
Jan 25, 2025

Conversation

ErBWs
Copy link
Contributor

@ErBWs ErBWs commented Jan 25, 2025

@ErBWs ErBWs reopened this Jan 25, 2025
@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 25, 2025

ohos 文件总是占满了选择区域,每次都忘记检查 pubspec.yaml 然后被默认勾选上传上来了

@Predidit
Copy link
Owner

现在关于页面的 dandanplay 条目下方会显示 APPID

你可以查看一下 macOS/iOS 产物的 APPID 是否已经被正确替换了吗

@Predidit Predidit merged commit 6a9f46b into Predidit:main Jan 25, 2025
9 of 12 checks passed
@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 25, 2025

显示了空 ID,这是正确的吗,至少替换应该是成功了

我在本地测试这个命令也可以看到 mortis 文件被正确替换了

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 25, 2025

@Predidit 噢好像有另一个问题,ohos 的 app-key 该怎么办

@Predidit
Copy link
Owner

Predidit commented Jan 25, 2025

不正确,应该有另一个ID

我看看哪里出了问题

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 25, 2025

那看上去可能有 github.secrets 解析问题

@ErBWs ErBWs deleted the fix-macos-sed branch January 25, 2025 02:24
@Predidit
Copy link
Owner

@ErBWs

检查一下现在的 CI

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 25, 2025

现在显示正常了

@Predidit
Copy link
Owner

@ErBWs

macOS 和 iOS 现在显示哪个id

@ErBWs
Copy link
Contributor Author

ErBWs commented Jan 25, 2025

github secrets 替换后的

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.

2 participants