Skip to content

fix: typecheck and fix errors in glob.py #210

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

Closed
wants to merge 1 commit into from

Conversation

ezyang
Copy link
Owner

@ezyang ezyang commented Mar 25, 2025

Stack from ghstack (oldest at bottom):

Typecheck codemcp/glob.py and fix errors. Typecheck ONLY this file (typecheck command takes a filename as argument).

3ffd8f2  (Base revision)
2f6e403  Add missing List[str] type annotation for result variable and import Any from typing
d29a90c  Add List[str] type annotation for items variable
4b4ea7f  Add List[str] type annotation for processed_items variable
3ada72b  Add List[str] type annotation for result variable in find function
164fff5  Replace unused dirnames variable with underscore to avoid warning
HEAD     Remove unused Any import

codemcp-id: 218-fix-typecheck-and-fix-errors-in-glob-py

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Mar 25, 2025
Typecheck codemcp/glob.py and fix errors. Typecheck ONLY this file (typecheck command takes a filename as argument).

```git-revs
3ffd8f2  (Base revision)
2f6e403  Add missing List[str] type annotation for result variable and import Any from typing
d29a90c  Add List[str] type annotation for items variable
4b4ea7f  Add List[str] type annotation for processed_items variable
3ada72b  Add List[str] type annotation for result variable in find function
164fff5  Replace unused dirnames variable with underscore to avoid warning
HEAD     Remove unused Any import
```

codemcp-id: 218-fix-typecheck-and-fix-errors-in-glob-py
ghstack-source-id: 0f83054
Pull-Request-resolved: #210
@ezyang ezyang closed this in f0326d4 Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant