Skip to content

Add protovalidate annotations to DeviceSpec#3

Merged
lberrymage merged 1 commit into
mainfrom
protovalidate-device-spec
Sep 17, 2025
Merged

Add protovalidate annotations to DeviceSpec#3
lberrymage merged 1 commit into
mainfrom
protovalidate-device-spec

Conversation

@lberrymage
Copy link
Copy Markdown
Member

These annotations are equivalent to the validation that bundletool itself performs in DeviceSpecParser.validateDeviceSpec() in bundletool 1.18.1. Adding protovalidate annotations makes it easier for protovalidate-using consumers of android-bundle to validate device specs.

The code for the upstream validation method can be found at https://github.com/google/bundletool/blob/1.18.1/src/main/java/com/android/tools/build/bundletool/device/DeviceSpecParser.java#L77.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 17, 2025

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped✅ passed✅ passedSep 17, 2025, 6:38 PM

These annotations are equivalent to the validation that bundletool
itself performs in DeviceSpecParser.validateDeviceSpec() in bundletool
1.18.1. Adding protovalidate annotations makes it easier for
protovalidate-using consumers of android-bundle to validate device
specs.

The code for the upstream validation method can be found at
https://github.com/google/bundletool/blob/1.18.1/src/main/java/com/android/tools/build/bundletool/device/DeviceSpecParser.java#L77.
@lberrymage lberrymage force-pushed the protovalidate-device-spec branch from 74d548f to 3bf5945 Compare September 17, 2025 18:38
@lberrymage lberrymage merged commit d0061d4 into main Sep 17, 2025
6 checks passed
@lberrymage lberrymage deleted the protovalidate-device-spec branch September 17, 2025 18:40
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.

1 participant