Skip to content
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

feature: 插件包管理支持对象存储模式,公共方法优化整合 (#2) #31

Conversation

ZhuoZhuoCrayon
Copy link
Member

issue #2

相关工作:

  • 将耦合的本地文件操作,基于Django Storage API 抽象,提供更灵活的替换文件源方式
  • 插件包上传 -> 制作 -> 导出 相关文件操作的改造
  • 对类似实现及字面量进行整合提取,并完善了utils的单元测试
  • 优化原有的逻辑

改造范围:

wecom-temp-0c7e56e641a5577b125751560d24dab4

基本思路:

wecom-temp-c445f05f0d6b92921ca9e0d9b91c0864

feature: 插件包管理支持对象存储模式,公共方法优化整合 (TencentBlueKing#2)
@ZhuoZhuoCrayon ZhuoZhuoCrayon self-assigned this Aug 24, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #31 (f090bbc) into V2.1.341rc_issue#2 (100d729) will increase coverage by 26.70%.
The diff coverage is n/a.

Impacted file tree graph

@@                   Coverage Diff                   @@
##           V2.1.341rc_issue#2      #31       +/-   ##
=======================================================
+ Coverage               59.00%   85.71%   +26.70%     
=======================================================
  Files                     390        1      -389     
  Lines                   25738        7    -25731     
=======================================================
- Hits                    15187        6    -15181     
+ Misses                  10551        1    -10550     
Impacted Files Coverage Δ
manage.py 85.71% <0.00%> (-14.29%) ⬇️
apps/node_man/views/debug.py
pipeline/builder/flow/__init__.py
apps/backend/components/collections/base.py
pipeline/core/flow/base.py
...n/periodic_tasks/clean_requests_tracker_records.py
apps/backend/components/collections/gse.py
version_log/models.py
pipeline/engine/health/zombie/heal.py
apps/backend/constants.py
... and 354 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 100d729...f090bbc. Read the comment docs.

@ZhuoZhuoCrayon ZhuoZhuoCrayon merged commit f9b4953 into TencentBlueKing:V2.1.341rc_issue#2 Aug 24, 2021
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