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

[agp] Upgrade AGP, Kotlin, Gradle, maven publish plugin #986

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

passy
Copy link
Member

@passy passy commented Apr 20, 2024

@passy passy changed the title [flipper] Update Flipper dependency [agp] Upgrade AGP, Kotlin, Gradle, maven publish plugin Apr 20, 2024
@passy passy force-pushed the pr986 branch 3 times, most recently from 8b19033 to 7d5f581 Compare April 25, 2024 16:59
@passy passy force-pushed the pr986 branch 2 times, most recently from d546726 to 2ba7aea Compare April 29, 2024 09:53
Pascal Hartig and others added 3 commits May 13, 2024 11:28
Summary:

The tests are not actually being run.

After upgrading the Gradle plugin on GitHub, it has become clear that these are running without the AP processing them. When processed, they raise some fairly obvious mistakes:

{P1221730610}

This is removing them for now so we can get the update and new release out.

Done using

```
pastry P1221730610 | rg -F '/src/' | awk '{print $1}' | cut -d ':' -f 1 | uniq | sed -e 's:/Users/realpassy/Projects/java/litho/::' | xargs hg rm -v
```

To restore the rests, let's do it one-by-one by doing `hg revert -r 5ac836c8ff21bb798ce1aba47a47857f93f3d47c^ -- litho-it/path/to/test.java`

Differential Revision: D56517990
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.

None yet

2 participants