Skip to content

build: Updated the obfuscation build configuration of libpag, now automatically including obfuscation in the AAR without the need for manual setup#3250

Merged
domchen merged 2 commits intoTencent:mainfrom
limuyang2:aar
Feb 2, 2026

Conversation

@limuyang2
Copy link
Contributor

  • Updated the obfuscation build configuration of libpag, now automatically including obfuscation in the AAR without the need for manual setup

@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.20%. Comparing base (dd74877) to head (1681570).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3250      +/-   ##
==========================================
+ Coverage   77.10%   77.20%   +0.09%     
==========================================
  Files         415      415              
  Lines       22040    22040              
  Branches     6307     6232      -75     
==========================================
+ Hits        16994    17015      +21     
+ Misses       3816     3812       -4     
+ Partials     1230     1213      -17     

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

release {
minifyEnabled true
proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.pro'
minifyEnabled false
Copy link
Collaborator

Choose a reason for hiding this comment

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

添加 consumerProguardFiles 是可以的,但是为啥要修改这里,两者作用是独立的哈
不要修改 libpag 原有的混淆逻辑

Copy link
Contributor Author

@limuyang2 limuyang2 Feb 2, 2026

Choose a reason for hiding this comment

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

这个项目本来就是开源库,release打包aar的时候,不需要混淆。将混淆规则带入consumerProguardFiles,由使用者来应用混淆(minifyEnabled

Copy link
Collaborator

Choose a reason for hiding this comment

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

那就把 debug 下面的也移除吧,后续 libpag 只提供混淆规则,不再处理混淆

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已处理

@domchen domchen merged commit 187bd07 into Tencent:main Feb 2, 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.

4 participants