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.

Sign up for GitHub

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

Jump to bottom

更新 GH Action #318

Merged
merged 5 commits into from
Feb 18, 2023
Merged

更新 GH Action #318

merged 5 commits into from
Feb 18, 2023

Conversation

sunxyw
Copy link
Member

@sunxyw sunxyw commented Feb 17, 2023

No description provided.

@sunxyw sunxyw added the workflow 有关工作流的更新 label Feb 17, 2023
@sunxyw sunxyw requested a review from crazywhalecc February 17, 2023 08:33
crazywhalecc
crazywhalecc requested changes Feb 17, 2023
View reviewed changes
Copy link
Member

@crazywhalecc crazywhalecc left a comment

Choose a reason for hiding this comment

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

PHP 版本应该使用 8.0 作为最宽适配，其次应该在部署环境时采用 --no-dev 参数，不需要 require-dev 的包安装。

@sunxyw sunxyw requested a review from crazywhalecc February 17, 2023 23:19
@sunxyw
Copy link
Member Author

sunxyw commented Feb 17, 2023

后续会对现有 workflow 流做少许优化

@sunxyw sunxyw merged commit 712b3e8 into main Feb 18, 2023
@sunxyw sunxyw deleted the update-gh-actions branch February 18, 2023 07:09
@sunxyw sunxyw mentioned this pull request Feb 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crazywhalecc crazywhalecc crazywhalecc approved these changes

Assignees
No one assigned
Labels
workflow 有关工作流的更新
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sunxyw @crazywhalecc