You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hugegraph.apache.org by "dosubot[bot] (via GitHub)" <gi...@apache.org> on 2024/05/29 07:14:31 UTC
Re: [I] [Question] A question about introducing black for code format [incubator-hugegraph-ai]
dosubot[bot] commented on issue #46:
URL: https://github.com/apache/incubator-hugegraph-ai/issues/46#issuecomment-2136699890
<!-- Greeting -->
@ChenZiHong-Gavin 你好! 我是Dosu,一个可以帮助你解决错误、回答问题并成为贡献者的机器人。我将在等待人类维护者的过程中协助你。
<!-- Answer -->
You should modify `pylint.yml` to include code formatting tasks as well. The provided script `style/code_format_and_analysis.sh` already supports running both BLACK and PYLINT together or separately based on the provided command-line arguments. This indicates that the workflow can be combined.
Here is an example of how you might modify `pylint.yml` to include BLACK:
```yaml
name: "Lint and Format"
on:
push:
branches:
- 'release-*'
pull_request:
jobs:
build:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ["3.9", "3.10"]
steps:
- uses: actions/checkout@v4
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v3
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install black pylint pytest
pip install -r ./hugegraph-llm/requirements.txt
pip install -r ./hugegraph-llm/llm_api/requirements.txt
pip install -r ./hugegraph-python-client/requirements.txt
- name: Run code format and analysis
run: |
chmod +x style/code_format_and_analysis.sh
./style/code_format_and_analysis.sh -b -p
```
This workflow will run both BLACK and PYLINT as specified in the `style/code_format_and_analysis.sh` script <sup>[[1]](https://github.com/apache/incubator-hugegraph-ai/blob/main/style/code_format_and_analysis.sh)</sup><sup>[[2]](https://github.com/apache/incubator-hugegraph-ai/blob/main/.github/workflows/pylint.yml)</sup>.
<!-- Dosu Comment Footer -->
<details open>
<summary>
#### Details
</summary>
**About Dosu**
This response is meant to be useful and save you time. It is **not meant to be a precise solution**, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
</details>
*To continue the conversation, mention @dosu.*
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For additional commands, e-mail: issues-help@hugegraph.apache.org