Skip to content

Switch build system to Hatchling which has much better Language Server support #55

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 18, 2025

Conversation

drisspg
Copy link
Contributor

@drisspg drisspg commented May 18, 2025

Stacked PRs:


Switch build system to Hatchling which has much better Language Server support

The call into setup.py feels like overkill as well since we aren't utilizing it for any real features. With this change I finally get good intellisense. I have found hatchling to be the best build system for python only projects and it is the top of recommendations for form PyPA

Screenshot 2025-05-18 at 9 58 10 AM

drisspg added a commit that referenced this pull request May 18, 2025
…r support

stack-info: PR: #55, branch: drisspg/stack/4
@drisspg drisspg force-pushed the drisspg/stack/4 branch from 44ed7f2 to e7f874e Compare May 18, 2025 16:52
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label May 18, 2025
drisspg added a commit that referenced this pull request May 18, 2025
…r support

stack-info: PR: #55, branch: drisspg/stack/4
@drisspg drisspg force-pushed the drisspg/stack/4 branch from e7f874e to c0763f7 Compare May 18, 2025 16:55
…r support

stack-info: PR: #55, branch: drisspg/stack/4
@drisspg drisspg force-pushed the drisspg/stack/4 branch from c0763f7 to bea4810 Compare May 18, 2025 16:55
@drisspg drisspg requested a review from oulgen May 18, 2025 16:57
@oulgen
Copy link
Contributor

oulgen commented May 18, 2025

@drisspg also FYI, i'm gonna setup diff train to bring helion into fbcode, so I will also write TARGETS file this week

@oulgen
Copy link
Contributor

oulgen commented May 18, 2025

Can you also add instructions to the README.md

@drisspg
Copy link
Contributor Author

drisspg commented May 18, 2025

So actually no other instructions are needed in the Readme, just install in editable pip install -e . or w/ dev requirements pip install -e .'[dev]' for all the reqs and you are good to. The default OSS vscode workflow where you select an interpreter now is able to jump to source for the editable install I site-packages. This means that you can be editing file in location X and go-to-def will take you to the source code on disc.

@drisspg drisspg merged commit b27e2cf into main May 18, 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.

4 participants