Skip to content

Conversation

wesm
Copy link
Contributor

@wesm wesm commented Aug 26, 2025

Without these, when you ask Claude to help with the Python extension (and the positron module in particular), it has to do some work to discover the right approach to run the unit tests etc., and it can go down dark alleys. These instructions can surely be made much better

@wesm wesm requested a review from seeM August 26, 2025 16:32
@wesm wesm force-pushed the claude-python-extension branch from ec201e3 to c6090db Compare August 26, 2025 16:46
seeM
seeM previously approved these changes Aug 27, 2025
Copy link
Contributor

@seeM seeM left a comment

Choose a reason for hiding this comment

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

Thanks @wesm for taking the initiative and setting up our Claude team infrastructure! I'm sure many of us have been experimenting and glad we now have a platform to share that.

For future: How do you handle permissions for bash commands like running tests? Should we consider shared settings files with well-scoped commands allowed?

Should we consider helper scripts e.g. one that runs ruff check, format check, and pyright? Perhaps another script that runs ruff check --fix and format? Etc.

@wesm
Copy link
Contributor Author

wesm commented Aug 29, 2025

Should we consider helper scripts e.g. one that runs ruff check, format check, and pyright? Perhaps another script that runs ruff check --fix and format? Etc.

I took a pass on this and added some helper scripts. Let me know if what I added is what you had in mind?

seeM
seeM previously approved these changes Sep 1, 2025
Copy link
Contributor

@seeM seeM left a comment

Choose a reason for hiding this comment

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

Looks good, thanks! The outline of helper scripts is what I had in mind. We can add the implementations separately if needed.

@wesm
Copy link
Contributor Author

wesm commented Sep 1, 2025

I guess I forgot to check the scripts in! I’ll rebase and add those and then merge

@wesm wesm force-pushed the claude-python-extension branch from 27c719d to 44b63c4 Compare September 1, 2025 18:49
@wesm wesm merged commit 5d27eb5 into main Sep 2, 2025
28 checks passed
@wesm wesm deleted the claude-python-extension branch September 2, 2025 11:02
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants