Skip to content

Conversation

@Rittzz
Copy link
Contributor

@Rittzz Rittzz commented Nov 5, 2022

Summary: See title. Left Windows out so it still compiles.

Test Plan:
Add a #fail below this line and build for various platforms and confirm it fails which proves the #ifdef was hit.

buck2 build xplat/langtech/tuna/cli:tuclixAndroid
buck2 build xplat/langtech/tuna/cli:tuclix

CI/CD for the rest.

Differential Revision: D41054824

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 5, 2022

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/88545

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 27fd850:
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 5, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Rittzz / name: Ian Graves (27fd850)

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D41054824

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D41054824

@qihqi qihqi self-requested a review November 8, 2022 23:00
…ytorch#88545)

Summary:
Pull Request resolved: pytorch#88545

See title.  Left Windows out so it still compiles.

Test Plan:
Add a `#fail` below [this line](https://fburl.com/code/p0mlhlw4) and build for various platforms and confirm it fails which proves the `#ifdef` was hit.

```
buck2 build xplat/langtech/tuna/cli:tuclixAndroid
buck2 build xplat/langtech/tuna/cli:tuclix
```

CI/CD for the rest.

Reviewed By: qihqi

Differential Revision: D41054824

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

This pull request was exported from Phabricator. Differential Revision: D41054824

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -f 'Landed internally'

(Initiating merge automatically since Phabricator Diff has merged, using force because this PR might not pass merge_rules.json but landed internally)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: The following mandatory check(s) failed (Rule superuser):

Dig deeper by viewing the failures on hud

Details for Dev Infra team Raised by workflow job

@izaitsevfb
Copy link
Contributor

/easycla

1 similar comment
@malfet
Copy link
Contributor

malfet commented Nov 11, 2022

/easycla

@malfet
Copy link
Contributor

malfet commented Nov 11, 2022

@pytorchbot merge -f "Landed internally already"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
…ytorch#88545)

Summary: See title.  Left Windows out so it still compiles.

Test Plan:
Add a `#fail` below [this line](https://fburl.com/code/p0mlhlw4) and build for various platforms and confirm it fails which proves the `#ifdef` was hit.

```
buck2 build xplat/langtech/tuna/cli:tuclixAndroid
buck2 build xplat/langtech/tuna/cli:tuclix
```

CI/CD for the rest.

Differential Revision: D41054824

Pull Request resolved: pytorch#88545
Approved by: https://github.com/qihqi
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.

6 participants