remove pre-commit from ci

This commit is contained in:
c9s 2022-06-21 15:27:48 +08:00
parent 06de735df0
commit c4ac2a23c4
No known key found for this signature in database
GPG Key ID: 7385E7E464CB0A54
2 changed files with 9 additions and 15 deletions

View File

@ -32,14 +32,6 @@ jobs:
restore-keys: | restore-keys: |
${{ runner.os }}-go- ${{ runner.os }}-go-
- uses: actions/cache@v2
with:
path: |
~/.cache/pip
key: ${{ runner.os }}-pip
restore-keys: |
${{ runner.os }}-pip
- name: Set up MySQL - name: Set up MySQL
run: | run: |
sudo /etc/init.d/mysql start sudo /etc/init.d/mysql start
@ -56,13 +48,6 @@ jobs:
with: with:
go-version: 1.18 go-version: 1.18
- name: Run pre-commit
run: |
pip install pre-commit
pre-commit install-hooks
pre-commit run markdownlint --files=README.md --verbose
pre-commit run detect-secrets --all-files --verbose
- name: Install Migration Tool - name: Install Migration Tool
run: go install github.com/c9s/rockhopper/cmd/rockhopper@v1.2.1 run: go install github.com/c9s/rockhopper/cmd/rockhopper@v1.2.1

View File

@ -22,6 +22,15 @@ If you find an issue to work on, you are welcome to open a PR with a fix.
### Making Changes ### Making Changes
Install pre-commit to check your changes before you commit:
pip install pre-commit
pre-commit install
pre-commit run markdownlint --files=README.md --verbose
pre-commit run detect-secrets --all-files --verbose
See <https://pre-commit.com/> for more details.
For new large features, such as integrating binance futures contracts, please propose a discussion first before you start working on it. For new large features, such as integrating binance futures contracts, please propose a discussion first before you start working on it.
For new small features, you could open a pull request directly. For new small features, you could open a pull request directly.