Skip to content

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat: 提交workflow配置 #13

wants to merge 5 commits into from

Conversation

junwense
Copy link

@junwense junwense commented May 14, 2025

#8

1.codecov token和ymal需要管理员配置下
2.llicense 这个还需要把现在代码都加上么

junwense added 2 commits May 19, 2025 10:02
1.修改ci go 版本
2.修改makefile里面的测试,新增e2e测试
Copy link
Contributor

@flycash flycash left a 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:
Copy link
Contributor

Choose a reason for hiding this comment

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

合理也打开,同样是针对 main 和 dev 分支

Comment on lines +41 to +47
- 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
Copy link
Contributor

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:
Copy link
Contributor

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
Copy link
Contributor

Choose a reason for hiding this comment

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

用 v4 了

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