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

Use BuildConfig instead of reading version from properties file #349

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

Conversation

daniel-shuy
Copy link

What does this PR do?

Fixes #288

Use com.github.gmazzo.buildconfig Gradle plugin (a plugin that generates BuildConfig like the Gradle android plugin, but for non-Android Java projects) to generate BuildConfig with project version at build time, instead of reading from pusher.properties at runtime, which fails with DiskReadViolation when running on Android with StrictMode enabled.

CHANGELOG

  • [CHANGED] Use BuildConfig instead of reading version from properties file

@daniel-shuy
Copy link
Author

Do I need to update the version in build.gradle to 2.4.5? Would be nice to have a CONTRIBUTING.md file for contributors

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.

PusherOptions.readVersionFromProperties android.os.strictmode.DiskReadViolation
2 participants