-
Notifications
You must be signed in to change notification settings - Fork 4
feat: 提交workflow配置 #13
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
把 script 的内容都挪过去 .script 里面
- 'docs/**' | ||
- '**.md' | ||
|
||
# pull_request: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
合理也打开,同样是针对 main 和 dev 分支
- name: Install Docker Compose | ||
run: | | ||
sudo apt-get update | ||
sudo apt-get install -y docker-compose | ||
|
||
- name: Start Docker service | ||
run: sudo service docker start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这些都不需要,action 内部本身就是支持 docker 的
ports: | ||
# 注意这里我映射为了 13306 端口 | ||
- "13306:3306" | ||
mysql8_test: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个文件放回去 .script 目录里面
build: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
用 v4 了
#8
1.codecov token和ymal需要管理员配置下
2.llicense 这个还需要把现在代码都加上么