Skip to content

fix: include template files in package installation #242

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

Open
wants to merge 3 commits into
base: gh/ezyang/185/base
Choose a base branch
from

Conversation

ezyang
Copy link
Owner

@ezyang ezyang commented Apr 13, 2025

Stack from ghstack (oldest at bottom):

codemcp/templates/ contains data files that aren't Python files per se, but they need to be installed properly so that they're available to be copied when we run codemcp init. Fix pyproject.toml so that we correctly include the data files. I think I want to recursively glob everything that isn't excluded by .gitignore, but if it's best practice to directly only package things that are Git versioned that would work too (not sure how you gonna do source tarballs that way though.)

3f886df  (Base revision)
d77a938  Add Hatchling configuration to include template files in package build
HEAD     Auto-commit format changes

codemcp-id: 253-fix-include-template-files-in-package-installation

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Apr 13, 2025
codemcp/templates/ contains data files that aren't Python files per se, but they need to be installed properly so that they're available to be copied when we run codemcp init. Fix pyproject.toml so that we correctly include the data files. I think I want to recursively glob everything that isn't excluded by .gitignore, but if it's best practice to directly only package things that are Git versioned that would work too (not sure how you gonna do source tarballs that way though.)

```git-revs
3f886df  (Base revision)
d77a938  Add Hatchling configuration to include template files in package build
HEAD     Auto-commit format changes
```

codemcp-id: 253-fix-include-template-files-in-package-installation
ghstack-source-id: 8d5e37f
Pull-Request-resolved: #242
[ghstack-poisoned]
ezyang added a commit that referenced this pull request Apr 14, 2025
codemcp/templates/ contains data files that aren't Python files per se, but they need to be installed properly so that they're available to be copied when we run codemcp init. Fix pyproject.toml so that we correctly include the data files. I think I want to recursively glob everything that isn't excluded by .gitignore, but if it's best practice to directly only package things that are Git versioned that would work too (not sure how you gonna do source tarballs that way though.)

```git-revs
3f886df  (Base revision)
d77a938  Add Hatchling configuration to include template files in package build
0066218  Auto-commit format changes
HEAD     Replace force-include with pattern-based include for template files
```

codemcp-id: 253-fix-include-template-files-in-package-installation
ghstack-source-id: 5ad7eed
Pull-Request-resolved: #242
[ghstack-poisoned]
ezyang added a commit that referenced this pull request Apr 14, 2025
codemcp/templates/ contains data files that aren't Python files per se, but they need to be installed properly so that they're available to be copied when we run codemcp init. Fix pyproject.toml so that we correctly include the data files. I think I want to recursively glob everything that isn't excluded by .gitignore, but if it's best practice to directly only package things that are Git versioned that would work too (not sure how you gonna do source tarballs that way though.)

```git-revs
3f886df  (Base revision)
d77a938  Add Hatchling configuration to include template files in package build
0066218  Auto-commit format changes
b518315  Replace force-include with pattern-based include for template files
HEAD     Simplify package configuration to only include what's necessary
```

codemcp-id: 253-fix-include-template-files-in-package-installation
ghstack-source-id: 4e685c8
Pull-Request-resolved: #242
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