Skip to content
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

Clean up wheel contents. #144

Merged
merged 1 commit into from
Mar 22, 2024
Merged

Clean up wheel contents. #144

merged 1 commit into from
Mar 22, 2024

Conversation

matz-e
Copy link
Member

@matz-e matz-e commented Mar 19, 2024

Fixes #143. Replaces setuptools, which seems very happy to just add
everything in the current directory into the source distribution with
hatch, which is much, much, much more reasonable. Also:

  • Delete now obsolete MANIFEST.in
  • Delete outdated, non-functional _debug.py
  • Move init.py, core/hoc, core/mod into neurodamus/data to properly compartmentalize it

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@matz-e matz-e force-pushed the neurodamus-only-wheel branch 2 times, most recently from 472867b to e3cced2 Compare March 20, 2024 15:35
@matz-e matz-e requested a review from WeinaJi March 20, 2024 15:41
@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@matz-e matz-e force-pushed the neurodamus-only-wheel branch 2 times, most recently from d81689c to fb9d035 Compare March 20, 2024 17:16
@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@WeinaJi
Copy link
Collaborator

WeinaJi commented Mar 21, 2024

Thanks for the improvements. I've learned new things about hatching.
Just a question: I ran hatch version on a local checkout of tag 3.1.1 and the version returned is 3.1.2.dev0 instead of 3.1.1 . Will we get the same kind of version on the wheel generated at new tags?

@matz-e
Copy link
Member Author

matz-e commented Mar 21, 2024

Thanks for the improvements. I've learned new things about hatching. Just a question: I ran hatch version on a local checkout of tag 3.1.1 and the version returned is 3.1.2.dev0 instead of 3.1.1 . Will we get the same kind of version on the wheel generated at new tags?

Hm, it should. If I checkout this PR, run git tag 123 and then hatchling version, I get the expected output of 123. Do you have steps I can reproduce?

@WeinaJi
Copy link
Collaborator

WeinaJi commented Mar 22, 2024

Thanks for the improvements. I've learned new things about hatching. Just a question: I ran hatch version on a local checkout of tag 3.1.1 and the version returned is 3.1.2.dev0 instead of 3.1.1 . Will we get the same kind of version on the wheel generated at new tags?

Hm, it should. If I checkout this PR, run git tag 123 and then hatchling version, I get the expected output of 123. Do you have steps I can reproduce?

Ah yes, you are right. In fact, I made a change in my local check out of the tag, that's why I got dev0. The versioning is ok.

@WeinaJi
Copy link
Collaborator

WeinaJi commented Mar 22, 2024

I also tag @ferdonline @jorblancoa @atemerev here, to inform them of the new location of the hoc/mod folders and init.py file after this PR is merged.

@bbpbuildbot

This comment has been minimized.

Fixes #143. Replaces `setuptools`, which seems very happy to just add
everything in the current directory into the source distribution with
`hatch`, which is much, much, much more reasonable. Also:

* Delete now obsolete `MANIFEST.in`
* Delete outdated, non-functional `_debug.py`
* Move `init.py` into `neurodamus/data` to properly compartmentalize it
@bbpbuildbot
Copy link

@matz-e matz-e merged commit 67022be into main Mar 22, 2024
6 checks passed
@matz-e matz-e deleted the neurodamus-only-wheel branch March 22, 2024 14:24
WeinaJi pushed a commit that referenced this pull request Oct 14, 2024
Fixes #143. Replaces `setuptools`, which seems very happy to just add
everything in the current directory into the source distribution with
`hatch`, which is much, much, much more reasonable. Also:

* Delete now obsolete `MANIFEST.in`
* Delete outdated, non-functional `_debug.py`
* Move `init.py`, `core/hoc`, `core/mod` into `neurodamus/data` to
properly compartmentalize it
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.

Wheel pollutes Python's site packages directory
3 participants