Skip to content

Conversation

@MeteorsLiu
Copy link
Contributor

No description provided.

@MeteorsLiu MeteorsLiu changed the title feat: reduce calling libclang mutiple times feat(internal/config): reduce calling libclang mutiple times Jun 5, 2025
@MeteorsLiu MeteorsLiu force-pushed the config/optimize-pkghfile branch from 591882a to e5616b5 Compare June 5, 2025 08:43
@MeteorsLiu MeteorsLiu force-pushed the config/optimize-pkghfile branch from 0aaa2d9 to ec3dfe8 Compare June 5, 2025 10:23

want: "",
},
// FIXME: substring bug
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Note is recommended to use "MARKER(uid): note body" format.

Details

lint 解释

这个lint结果提示在注释中使用“MARKER(uid): note body”格式是一个建议。这意味着在编写注释时,应该遵循这种特定的格式来提高代码的可读性和一致性。

错误用法

以下是一个错误的示例,展示了不正确的注释格式:

// 这是一个错误的注释格式

正确用法

以下是一个正确的示例,展示了符合建议的注释格式:

// MARKER(12345): 这是一个正确的注释格式

💡 以上内容由 AI 辅助生成,如有疑问欢迎反馈交流


want: "testdata/hfile",
},
// FIXME: absolute path
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Note is recommended to use "MARKER(uid): note body" format.

Details

lint 解释

这个lint结果提示在注释中使用“MARKER(uid): note body”格式是一个推荐的做法。这意味着在编写注释时,应该遵循这种特定的格式来提高代码的可读性和一致性。

错误用法

以下是一个错误的示例,展示了不正确的注释格式:

// 这是一个错误的注释格式

正确用法

以下是一个正确的示例,展示了符合推荐格式的注释:

// MARKER(12345): 这是一个正确的注释格式

💡 以上内容由 AI 辅助生成,如有疑问欢迎反馈交流

@codecov
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.38%. Comparing base (8bb3596) to head (cc75254).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #423   +/-   ##
=======================================
  Coverage   84.38%   84.38%           
=======================================
  Files          27       27           
  Lines        2728     2728           
=======================================
  Hits         2302     2302           
  Misses        385      385           
  Partials       41       41           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@qiniu-x
Copy link

qiniu-x bot commented Jun 6, 2025

[Git-flow] Hi @MeteorsLiu, There are some suggestions for your information:


Rebase suggestions

  • Following commits seems generated via git merge

    Merge branch 'main' of https://github.com/goplus/llcppg into trie

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

  • Following commits have duplicated messages

    fix: mm output parse

    fix: mm output parse

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    Merge branch 'trie' into config/optimize-pkghfile

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    add test

    merge

    merge

Which seems insignificant, recommend to use git rebase command to reorganize your PR.

For other git-flow instructions, recommend refer to these examples.

Details

If you have any questions about this comment, feel free to raise an issue here:

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.

1 participant