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.

Sign up for GitHub

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

Jump to bottom

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

Draft

m1ga wants to merge 1 commit into master from productionLogs

Contributor

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


          chore: filter out logs in production build

61bea6a

hansemannn changed the title ~~chore: filter out logs in production build~~ chore(android): filter out logs in production build

m1ga added improvement BREAKING CHANGES ⚠️ labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BREAKING CHANGES ⚠️ improvement