Skip to content

use cobra command for llcppg - #582

Open
tsingbx wants to merge 9 commits into
xgo-dev:mainfrom
tsingbx:cpp
Open

use cobra command for llcppg#582
tsingbx wants to merge 9 commits into
xgo-dev:mainfrom
tsingbx:cpp

Conversation

@tsingbx

@tsingbxtsingbx commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

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

@fennoai

This comment was marked as resolved.

Comment threadcmd/internal/base/base.go Outdated
Comment threadcmd/internal/base/base.go Outdated
Comment threadcmd/internal/base/pass.go Outdated
Comment threadcmd/internal/base/pass.go Outdated
Comment threadcmd/internal/gencfg/gencfg.go Outdated
Comment threadcmd/internal/gencfg/gencfg.go Outdated
@codecov

codecovBot commented Nov 5, 2025

Copy link
Copy Markdown

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 linesPatch %Lines
cmd/internal/base/base.go0.00%44 Missing ⚠️
cmd/internal/genpkg/genpkg.go0.00%38 Missing ⚠️
config/parse.go18.18%35 Missing and 1 partial ⚠️
cmd/internal/gencfg/gencfg.go0.00%28 Missing ⚠️
config/config.go0.00%15 Missing ⚠️
cmd/internal/gencfg/flags.go0.00%7 Missing ⚠️
cmd/internal/gensig/gensig.go0.00%5 Missing ⚠️
cmd/internal/gensym/gensym.go0.00%5 Missing ⚠️
cmd/internal/version/version.go0.00%4 Missing ⚠️
cmd/internal/genpkg/flags.go0.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
FlagBASE (7955d98)HEAD (e847864)
32
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
tsingbxforce-pushed the cpp branch 2 times, most recently from 5145e64 to a7b73c0CompareNovember 6, 2025 01:21
@tsingbx
tsingbxforce-pushed the cpp branch 3 times, most recently from c8db74a to e1b7ce5CompareNovember 6, 2025 02:31
@tsingbx
tsingbxforce-pushed the cpp branch 2 times, most recently from 284c481 to 1173b10CompareNovember 6, 2025 05:24
@tsingbxtsingbx changed the title use cobra command for llcppguse cobra command for llcppgxNov 6, 2025
@tsingbxtsingbx changed the title use cobra command for llcppgxuse cobra command for llcppgNov 6, 2025
@tsingbx
tsingbxforce-pushed the cpp branch 4 times, most recently from cd461ba to c510c22CompareNovember 6, 2025 06:30
@tsingbx
tsingbx marked this pull request as ready for review November 6, 2025 07:23
Comment threadcmd/internal/base/base.go Outdated
Comment threadcmd/internal/genpkg/genpkg.go Outdated
Comment threadcmd/internal/genpkg/genpkg.go Outdated
Comment threadcmd/llcppgx/main_app.gox Outdated
Comment threadcmd/llcppgx/main_app.gox Outdated
Comment threadcmd/llcppgx/main_app.gox Outdated
Comment threadcmd/llcppgx/main_app.gox Outdated
Comment threadcmd/internal/gencfg/flags.go Outdated
Comment threadconfig/parse.go
@fennoai

This comment was marked as resolved.

@tsingbx
tsingbxforce-pushed the cpp branch 2 times, most recently from 23cc343 to 5b75f77CompareNovember 6, 2025 08:17
@tsingbx
tsingbxforce-pushed the cpp branch 2 times, most recently from e847864 to fb56e76CompareNovember 7, 2025 07:55
Sign up for freeto 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

@tsingbx