Skip to content

Removing ignore for *.json files in gitignore. #1019

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
Mar 10, 2025

Conversation

weinbe58
Copy link
Member

The reason is that Hatchling will exclude all JSON files included in the source directory if git ignores it for some reason. NOTE: I tried a few other options to get this to work without updating the git ignore:

  1. I updated the patterns for including files in a few ways
  2. I tried to force-include them explicitly by path.

None of these worked, I checked by running uv build and inspecting the contects of the sdist directly. every time the config folder was not there. Only after updating the gitignore did it work.

@weinbe58 weinbe58 linked an issue Mar 10, 2025 that may be closed by this pull request
Copy link
Contributor

github-actions bot commented Mar 10, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-03-10 15:18 UTC

Copy link
Contributor

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8178 7342 90% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 57172a1 by action🐍

Copy link

codecov bot commented Mar 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@weinbe58 weinbe58 requested a review from johnzl-777 March 10, 2025 14:40
@weinbe58 weinbe58 merged commit 77d046a into main Mar 10, 2025
8 checks passed
@weinbe58 weinbe58 deleted the phil/add-config-to-package branch March 10, 2025 15:17
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.

Configuration json files not installed with bloqade-analog
2 participants