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

Add missing API versions for Roboelectric #972

Closed
wants to merge 19 commits into from

Conversation

vicmns
Copy link

@vicmns vicmns commented Jan 30, 2023

Adding missing APIS 32 and 33 so they can be resolved by the OkBuck build system

Description:
Adding missing APIs levels to Roboelectric manager.

Adding missing APIS 32 and 33 so they can be resolved by the OkBuck build system
@CLAassistant
Copy link

CLAassistant commented Jan 30, 2023

CLA assistant check
All committers have signed the CLA.

@tyvsmith
Copy link
Member

Please update TestExtension.java to the latest preinstrumented verison while we're at it (same PR or separate).

Also, it would be good to add a CustomAPI support so we can update this without a PR, but we don't need to block on making that happen now.

@vicmns vicmns force-pushed the bump_roboelectric_versions branch 7 times, most recently from 57b0727 to ecdeef0 Compare January 30, 2023 23:50
@vicmns vicmns force-pushed the bump_roboelectric_versions branch 8 times, most recently from d278d1e to dd9f3d5 Compare February 8, 2023 00:44
Bump preinstrumented version to 4 as found on robo src code.

Run okbuck before performing CI tasks to update references
@vicmns vicmns force-pushed the bump_roboelectric_versions branch from dd9f3d5 to dafaa7f Compare February 8, 2023 00:51
@hvvikram hvvikram force-pushed the master branch 3 times, most recently from b7ce0ff to 815b968 Compare February 21, 2023 17:20
@tyvsmith
Copy link
Member

Is this stalled?

hvvikram added 2 commits May 31, 2023 10:21
* feat: Add support for exporting raw dependencies

plugin can accept two project parameters
* exportDependencies: to enable exporting of raw dependencies
* exportDependenciesFile: Name with optional relative path from the root of the project where the dependencies will be exported ( default: .okbuck/raw-deps )

Supports exporting raw dependencies in JSON format.

```
./gradlew okbuck -PexportDependencies=true -PexportDependenciesFile=.okbuck/raw-deps.json
```

* Moved configurations to OKBuckExtension and handle excludeRule to use group:module when both are not blank

* * Addressed review comments
* Ran google formatter
* Autoclose BufferedWriter to flush the contents and close
hvvikram and others added 8 commits May 31, 2023 10:21
* Add config cleanCacheDir which controls whether to delete the entire cache dir or only the existing dependency rules files, before generating the dependency rule files

* Switched to parallel delete

* Use TrueFileFilter for directory filter instead of null
@vicmns
Copy link
Author

vicmns commented May 31, 2023

Kind off, I'm still not able to get CI to resolve this artifact....

FAILURE com.uber.okbuck.example.empty.EmptyLibraryUnitTest addition_isCorrect: Path is not a file: /home/runner/work/okbuck/okbuck/buck-out/gen/.okbuck/workspace/robolectric/robolectric_cache/android-all-instrumented-8.1.0-robolectric-4611349-i3.jar

vicmns and others added 4 commits May 31, 2023 10:43
Adding missing APIS 32 and 33 so they can be resolved by the OkBuck build system
Bump preinstrumented version to 4 as found on robo src code.

Run okbuck before performing CI tasks to update references
@gpolak
Copy link
Contributor

gpolak commented Dec 16, 2023

We should abandon this PR as it's superseded by #983

@gpolak gpolak closed this Dec 16, 2023
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.

5 participants