Skip to content

refactor: actions - #51

Draft
MeteorsLiu wants to merge 23 commits into
xgo-dev:mainfrom
MeteorsLiu:refactor-actions
Draft

refactor: actions#51
MeteorsLiu wants to merge 23 commits into
xgo-dev:mainfrom
MeteorsLiu:refactor-actions

Conversation

@MeteorsLiu

@MeteorsLiuMeteorsLiu commented Apr 11, 2025

Copy link
Copy Markdown
Contributor

This PR is provided solely as a reference for refactoring, which is unnecessary to be reviewed
The refactoring process will be split into multiple PRs for review.

@codecov

codecovBot commented Apr 11, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 22.84382% with 331 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.12%. Comparing base (4420420) to head (a5372d2).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
internal/actions/api.go0.00%244 Missing ⚠️
internal/actions/env/env.go14.28%48 Missing ⚠️
internal/actions/actions.go26.08%17 Missing ⚠️
...rnal/actions/parser/mappedversion/mappedversion.go63.15%7 Missing ⚠️
internal/actions/mappingtable/versions.go86.84%4 Missing and 1 partial ⚠️
internal/actions/text.go0.00%4 Missing ⚠️
internal/actions/tag/tag.go80.00%2 Missing and 1 partial ⚠️
internal/actions/parser/prefix/prefix.go91.30%1 Missing and 1 partial ⚠️
cmd/llpkgstore/internal/verification.go0.00%1 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #51 +/- ##
==========================================
+ Coverage 40.85% 42.12% +1.26% 
==========================================
Files 33 38 +5 Lines 1745 1842 +97 ==========================================
+ Hits 713 776 +63 - Misses 947 979 +32 - Partials 85 87 +2 

☔ 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

qiniu-xBot commented Apr 14, 2025

Copy link
Copy Markdown
Contributor

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


Rebase suggestions

  • Following commits seems generated via git merge

    Merge remote-tracking branch 'llpkg/main' into refactor-actions

  • Following commits have duplicated messages

    chore: add comment

    chore: add comment

    chore: add mappedversion test

    chore: add mappedversion test

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

@MeteorsLiu