Skip to content

fix headers attribute of cxx_test in OSS #8662

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

Merged
merged 3 commits into from
Feb 25, 2025
Merged

Conversation

swolchok
Copy link
Contributor

We forgot to make header paths work for cxx_test headers.

Test Plan: buck2 build //runtime/kernel/test:operator_registry_test

[ghstack-poisoned]
[ghstack-poisoned]
@swolchok
Copy link
Contributor Author

swolchok commented Feb 25, 2025

Copy link

pytorch-bot bot commented Feb 25, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 26 Pending

As of commit 146cfa6 with merge base 6cb5c1a (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 25, 2025
swolchok added a commit that referenced this pull request Feb 25, 2025
We forgot to make header paths work for cxx_test headers.

Test Plan: buck2 build //runtime/kernel/test:operator_registry_test

ghstack-source-id: a1707b8
ghstack-comment-id: 2680189455
Pull Request resolved: #8662
@swolchok swolchok requested review from mergennachin, larryliu0820, dbort and GregoryComer and removed request for mergennachin February 25, 2025 03:04
@swolchok swolchok added the release notes: build Changes related to build, including dependency upgrades, build flags, optimizations, etc. label Feb 25, 2025
Base automatically changed from gh/swolchok/285/head to main February 25, 2025 03:05
[ghstack-poisoned]
@swolchok swolchok merged commit fcb40f1 into main Feb 25, 2025
46 of 47 checks passed
@swolchok swolchok deleted the gh/swolchok/286/head branch February 25, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: build Changes related to build, including dependency upgrades, build flags, optimizations, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants