Skip to content

Conversation

@tsingbx
Copy link
Contributor

@tsingbx tsingbx commented Nov 5, 2025

llcppg has too many features
the new llcppg, the llcppgx, purely user-oriented.

The usage of the llcppgx command is simplified as follows:

  1. llcppgx init libcjson Initializes libcjson
  2. llcppgx gensym Generates and edits the symbol table
  3. llcppgx genpkg -mod "github.com/goplus/cjson" Generates a Go package

The number of command-line flags is reduced.

@tsingbx tsingbx requested a review from aofei November 5, 2025 09:34
@tsingbx tsingbx marked this pull request as draft November 5, 2025 09:34
@gemini-code-assist

This comment was marked as outdated.

gemini-code-assist[bot]

This comment was marked as outdated.

@xgopilot

This comment was marked as resolved.

@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 4.06091% with 189 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.44%. Comparing base (7955d98) to head (e847864).

Files with missing lines Patch % Lines
cmd/internal/base/base.go 0.00% 44 Missing ⚠️
cmd/internal/genpkg/genpkg.go 0.00% 38 Missing ⚠️
config/parse.go 18.18% 35 Missing and 1 partial ⚠️
cmd/internal/gencfg/gencfg.go 0.00% 28 Missing ⚠️
config/config.go 0.00% 15 Missing ⚠️
cmd/internal/gencfg/flags.go 0.00% 7 Missing ⚠️
cmd/internal/gensig/gensig.go 0.00% 5 Missing ⚠️
cmd/internal/gensym/gensym.go 0.00% 5 Missing ⚠️
cmd/internal/version/version.go 0.00% 4 Missing ⚠️
cmd/internal/genpkg/flags.go 0.00% 3 Missing ⚠️
... and 2 more

❗ There is a different number of reports uploaded between BASE (7955d98) and HEAD (e847864). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (7955d98) HEAD (e847864)
3 2
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #582      +/-   ##
==========================================
- Coverage   84.19%   78.44%   -5.76%     
==========================================
  Files          27       38      +11     
  Lines        2746     2500     -246     
==========================================
- Hits         2312     1961     -351     
- Misses        391      496     +105     
  Partials       43       43              

☔ 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.

@tsingbx tsingbx force-pushed the cpp branch 2 times, most recently from 5145e64 to a7b73c0 Compare November 6, 2025 01:21
@tsingbx tsingbx force-pushed the cpp branch 3 times, most recently from c8db74a to e1b7ce5 Compare November 6, 2025 02:31
@tsingbx tsingbx requested review from visualfc and xushiwei November 6, 2025 03:38
@tsingbx tsingbx force-pushed the cpp branch 2 times, most recently from 284c481 to 1173b10 Compare November 6, 2025 05:24
@tsingbx tsingbx changed the title use cobra command for llcppg use cobra command for llcppgx Nov 6, 2025
@tsingbx tsingbx changed the title use cobra command for llcppgx use cobra command for llcppg Nov 6, 2025
@tsingbx tsingbx force-pushed the cpp branch 4 times, most recently from cd461ba to c510c22 Compare November 6, 2025 06:30
@tsingbx tsingbx marked this pull request as ready for review November 6, 2025 07:23
@xgopilot

This comment was marked as resolved.

@tsingbx tsingbx force-pushed the cpp branch 2 times, most recently from 23cc343 to 5b75f77 Compare November 6, 2025 08:17
@tsingbx tsingbx force-pushed the cpp branch 2 times, most recently from e847864 to fb56e76 Compare November 7, 2025 07:55
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