8000 Add ../pytorch-nightly to Pyre optional_search_path by yf225 · Pull Request #36 · pytorch-labs/helion · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add ../pytorch-nightly to Pyre optional_search_path #36

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 1 commit into from
May 13, 2025

Conversation

yf225
Copy link
Contributor
@yf225 yf225 commented May 13, 2025

On local development, I have the PyTorch nightly pip install'ed into the ../pytorch-nightly folder for easier file access. Would like to add this search path to Pyre config as well so that I can run lint check locally.

@yf225 yf225 requested review from jansel and oulgen May 13, 2025 20:42
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label May 13, 2025
@@ -10,7 +10,7 @@
"source": "examples"
}
],
"optional_search_path": ["../pytorch", "../pytorch-hg"],
"optional_search_path": ["../pytorch", "../pytorch-hg", "../pytorch-nightly"],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i wish there was a better way to do this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes would be awesome if this can be taken from an env var or something..

@yf225 yf225 merged commit 57b631d into pytorch-labs:main May 13, 2025
4 checks passed
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 Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0