8000 [sample] Remove Flipper integration temporarily by passy · Pull Request #985 · facebook/litho · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[sample] Remove Flipper integration temporarily #985

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

Closed
wants to merge 1 commit into from
Closed

Conversation

passy
Copy link
Member
@passy passy commented Apr 19, 2024

@facebook-github-bot
Copy link
Contributor

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@passy has updated the pull request. You must reimport the pull request before landing.

facebook-github-bot pushed a commit that referenced this pull request Jun 20, 2024
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`

 ---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/litho/pull/988).
* #986
* #985
* __->__ #988

Pull Request resolved: #988

Reviewed By: adityasharat

Differential Revision: D58789067

Pulled By: passy

fbshipit-source-id: fb1aacd92dc88e0fb4d7ad4dc4c918396958fc3f
@facebook-github-bot
Copy link
Contributor

@passy has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@passy merged this pull request in 90df873.

passy added a commit that referenced this pull request Jul 17, 2024
passy added a commit that referenced this pull request Jul 18, 2024
passy added a commit that referenced this pull request Jul 18, 2024
facebook-github-bot pushed a commit that referenced this pull request Jul 19, 2024
Summary:
Original commit changeset: 90df873

Pull Request resolved: #1006

Reviewed By: adityasharat

Differential Revision: D59860796

Pulled By: passy

fbshipit-source-id: e7cbe5621a7aa1ef54d782f70ddc1cf2438d5b03
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.

2 participants
0