Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(android): filter out logs in production build #14032

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

m1ga
Copy link
Contributor

@m1ga m1ga commented May 10, 2024

Currently console.log(), console.warn() and console.error() are displayed in production builds. This PR will remove that.

Ti.API.info("#### api info")
Ti.API.warn("#### api warn")
Ti.API.error("#### api error")

console.log("#### console log")
console.warn("#### console warn")
console.error("#### console error")

Test

  • run the code above in a dev/device build and you see all the log
  • build for production and install the APK
  • run adb logcat | grep -i "####" to get all the logs

Todo
Still searching for the Ti.API.info stuff. That shouldn't be there either! I thought it was already blocked in the past

@hansemannn hansemannn changed the title chore: filter out logs in production build chore(android): filter out logs in production build May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant