build: add CSV-Analyzer

martyr-deepin / linglong-hub

Deprecated. Moved to https://github.com/linglongdev

https://github.com/linglongdev

GNU Lesser General Public License v3.0

41 stars 45 forks source link

build: add CSV-Analyzer #2205

Closed wjyrich closed 4 months ago

wjyrich commented 4 months ago

It can plot value vs line number/ time value (for only known log formats), and value vs another value scatter graphs.

Log: add software name--CSV-Analyzer

kamiyadm commented 4 months ago

patch应用失败了

wjyrich commented 4 months ago

/check build

wjyrich commented 4 months ago

/check build

wjyrich commented 4 months ago

/check build

deepin-bot[bot] commented 4 months ago

Deepin Obs Bot Starting find obs webhook event and retrigger!

deepin-ci-robot commented 4 months ago

deepin pr auto review

关键摘要:

在patches/0001-install.patch中，删除和插入的行顺序可能需要调整，以保持代码的一致性和可读性。
在CsvAnalyzer.pro文件中，else块中的注释应该放在else之后，而不是之前。
target.path的赋值应该在else块中，以确保在所有平台上都使用正确的路径。
desktop.files和desktop.path的赋值应该在desktop.files之前，以确保顺序正确。
icons.files和icons.path的赋值应该在icons.files之前，以确保顺序正确。
缺少对新增代码行的解释注释，应该说明为什么需要这些更改。

是否建议立即修改: 是

deepin-ci-robot commented 4 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kamiyadm, wjyrich

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/linglong-hub/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment