Add additional Android requirements to README#68
Add additional Android requirements to README#68jack-berg merged 3 commits intoopen-telemetry:mainfrom
Conversation
| > built on debug, you must use AGP 8.3.0+ and set the `android.useFullClasspathForDexingTransform` | ||
| > property in `gradle.properties` to `true` to ensure desugaring runs properly. For the full | ||
| > context for this workaround, please see | ||
| > [this issue](https://issuetracker.google.com/issues/230454566#comment18). |
There was a problem hiding this comment.
This renders weird. Can you remove the > quoting and change > Android Requirements to ## Android Requirements?
|
|
||
| > Android Requirements | ||
| > | ||
| > If you are using this on Android and your project's minSdk is lower than 26, you must enable |
There was a problem hiding this comment.
Can we just link to the core repo section which talks about desugaring? https://github.com/open-telemetry/opentelemetry-java/blob/main/VERSIONING.md#language-version-compatibility
The "minSdk is lower than 26" requirement is the type of thing what will become out of date and I'm not really sure where 26 comes from.
There was a problem hiding this comment.
26 is Android API version code. It's pretty impenetrable unless you're deep in that ecosystem.
I'll make a quick reference to desugaring here and point to the opentelementry-android repo for more details so we can isolate the details in one place at least.
I'll put up a revision in the next couple days.
|
Updated. I linked to the PR instead of the README for details in case the README changes. I can change that if y'all want. |
|
Looks like the build is failing for something unrelated to this PR, which should be fixed in #71. After that is merged, we can merge main into this branch and merge. |
Modify README to reference an additional requirement for Android projects that support Android versions that require desugaring. Details are on this Slack thread.
I can't find a style guide that tells me how this modification should be formatted, so please let me know if this doesn't work and I'll fix it up!