-
Notifications
You must be signed in to change notification settings - Fork 416
Merge Master #7844
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: dev_multi_tenant
Are you sure you want to change the base?
Merge Master #7844
Conversation
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
| runs-on: ubuntu-22.04 | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - name: Set up Python 3.6 | ||
| uses: actions/setup-python@v2 | ||
| with: | ||
| python-version: 3.6 | ||
| - name: Install dependencies | ||
| - uses: actions/checkout@v4 | ||
| - name: Install system dependencies (clang) | ||
| run: | | ||
| python -m pip install --upgrade pip | ||
| pip install flake8 black | ||
| sudo apt-get update | ||
| sudo apt-get install -y clang libssl-dev zlib1g-dev libbz2-dev libreadline-dev libsqlite3-dev libffi-dev | ||
| - name: Install pyenv | ||
| run: | | ||
| curl -L https://github.com/pyenv/pyenv-installer/raw/master/bin/pyenv-installer | bash | ||
| - name: Install system dependencies (python) | ||
| run: | | ||
| export PYENV_ROOT="$HOME/.pyenv" | ||
| export PATH="$PYENV_ROOT/bin:$PATH" | ||
| eval "$(pyenv init --path)" | ||
| CC=clang pyenv install 3.6.12 -v | ||
| sudo ln -sf $PYENV_ROOT/versions/3.6.12/bin/python3.6 /usr/local/bin/python3.6 | ||
| python3.6 -m pip install --upgrade pip | ||
| python3.6 -m pip install flake8==3.9.2 black==21.12b0 | ||
| - name: Lint with flake8 | ||
| run: | | ||
| flake8 | ||
| python3.6 -m flake8 | ||
| - name: Format with black | ||
| run: | | ||
| black . | ||
| python3.6 -m black . |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 7 months ago
To fix the issue, we will add a permissions block at the root of the workflow file. This block will specify the minimal permissions required for the workflow to function. Based on the workflow's steps, it does not appear to require any write permissions, so we will set contents: read as the permission. This ensures that the workflow has only read access to the repository contents.
-
Copy modified lines R6-R8
| @@ -5,2 +5,5 @@ | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| on: |
.github/workflows/unittest.yml
Outdated
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 7 months ago
To fix the issue, add a permissions block at the root of the workflow file. This block will define the minimal permissions required for the workflow. Based on the tasks in the workflow, the following permissions are needed:
contents: readfor accessing the repository's code.actions: writefor uploading coverage reports to Codecov.
The permissions block should be added at the top level of the workflow file, ensuring it applies to all jobs in the workflow.
-
Copy modified lines R3-R6
| @@ -2,2 +2,6 @@ | ||
|
|
||
| permissions: | ||
| contents: read | ||
| actions: write | ||
|
|
||
| on: |
.github/workflows/unittest.yml
Outdated
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
Uses Step
Codecov Report❌ Patch coverage is Please upload reports for the commit 785d0ff to get more accurate results. Additional details and impacted files@@ Coverage Diff @@
## dev_multi_tenant #7844 +/- ##
====================================================
- Coverage 58.99% 58.62% -0.37%
====================================================
Files 655 665 +10
Lines 34599 35580 +981
====================================================
+ Hits 20410 20858 +448
- Misses 14189 14722 +533
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
# Reviewed, transaction id: 52786
feat: 请求上下文返回上报地址 --story=125100805
# Reviewed, transaction id: 52923
fix: 修复读取数据格式错误 --story=125100805
# Reviewed, transaction id: 53227
refactor: 增加支持继承属性的trace装饰器和上下文管理器 --story=126421543
# Reviewed, transaction id: 53421
refactor: 网关和saas接口补充trace入口 --story=126421543
# Reviewed, transaction id: 53497
refactor: 修复测试用例没有traceprovider的问题 --story=126421543
# Reviewed, transaction id: 53635
# Reviewed, transaction id: 53649
# Reviewed, transaction id: 53688
feat: 任务模板支持webhook --story=126514126
# Reviewed, transaction id: 53704
# Reviewed, transaction id: 53724
# Reviewed, transaction id: 53724
…BlueKing/bk-sops into feature_template_webhook
# Reviewed, transaction id: 53789
refactor: 删除postcss依赖 #ignore
# Reviewed, transaction id: 53872
# Reviewed, transaction id: 53999
feat: 增加接口获取任务执行的有效时长 --story=129548331
# Reviewed, transaction id: 69902
# Reviewed, transaction id: 70032
fix: 修复网关api请求变量结构转换问题 --story=152536452
feat: 网关MCP Server对返回的信息进行过滤 --story=129445652
feat: 增加判断是否来源于MCP的逻辑 --story=129445652
feat: 增加判断是否来源于MCP的逻辑 --story=129445652
# Reviewed, transaction id: 70745
# Reviewed, transaction id: 70836
# Reviewed, transaction id: 71286
feat: apigw包下添加get_task_node_log和get_task_plugin_log两个接口--story=130004643
fix: 修复apigw下两个接口get_task_node_log和get_task_plugin_log两个接口鉴权问题--story=130004643
refactor: 清理weixin目录 --story=130071508
refactor: 优化api-resources.yml中网关API的声明 --story=130158303
# Reviewed, transaction id: 72225
# Reviewed, transaction id: 72281
# Reviewed, transaction id: 72282
# Reviewed, transaction id: 72359
No description provided.