Skip to content

Split the pagx convert command into import and export#3347

Merged
shlzxjp merged 8 commits intoTencent:mainfrom
OnionsYu:feature/onionsyu_cli_import_export
Mar 30, 2026
Merged

Split the pagx convert command into import and export#3347
shlzxjp merged 8 commits intoTencent:mainfrom
OnionsYu:feature/onionsyu_cli_import_export

Conversation

@OnionsYu
Copy link
Copy Markdown
Contributor

@OnionsYu OnionsYu commented Mar 27, 2026

Summary

  • Split the single convert command into separate import and export subcommands for clearer CLI semantics
  • export converts PAGX to other formats (SVG), import converts other formats (SVG) to PAGX
  • Updated tests to cover both new commands independently

Test plan

  • Existing convert tests migrated to import/export tests
  • Verify pagx export --input file.pagx produces SVG output
  • Verify pagx import --input file.svg produces PAGX output
  • Verify error handling for missing input and unsupported formats

🤖 Generated with Claude Code

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 96.30542% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.59%. Comparing base (0538ea0) to head (661ecfa).

Files with missing lines Patch % Lines
src/cli/CommandExport.cpp 91.30% 2 Missing and 6 partials ⚠️
src/cli/CommandImport.cpp 91.76% 2 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3347      +/-   ##
==========================================
+ Coverage   75.19%   75.59%   +0.39%     
==========================================
  Files         509      510       +1     
  Lines       35211    35435     +224     
  Branches    11270    11283      +13     
==========================================
+ Hits        26478    26787     +309     
+ Misses       6428     6355      -73     
+ Partials     2305     2293      -12     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@OnionsYu OnionsYu force-pushed the feature/onionsyu_cli_import_export branch from c01ec3d to 64a480a Compare March 27, 2026 06:19
@shlzxjp shlzxjp merged commit f456817 into Tencent:main Mar 30, 2026
9 checks passed
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.

3 participants