Skip to content

setup.py: fix or kill #105

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
drbenvincent opened this issue Dec 1, 2022 · 2 comments · Fixed by #156
Closed

setup.py: fix or kill #105

drbenvincent opened this issue Dec 1, 2022 · 2 comments · Fixed by #156
Labels
devops DevOps related

Comments

@drbenvincent
Copy link
Collaborator

Changes/improvements to setup.py from @juanitorduz in #79 ended up breaking the pypi build when I released version 0.0.4. I quickly fixed this with #99 by rolling back changes to setup.py.

At the moment, it's not clear if the error was coming from how the version is dealt with (line below works fine and is good)

exec(open("causalpy/version.py").read())

or changes to how requirements files were being read in and dealt with.

Thoughts:

  • Can we actually entirely remove setup.py and just have pyproject.toml?
  • If not, then:
@drbenvincent drbenvincent added the devops DevOps related label Dec 1, 2022
@juanitorduz
Copy link
Collaborator

juanitorduz commented Dec 1, 2022

I'm not very experienced in the subject but I j it about a very cool project ;) Wolt is maintaining a pretty modern python package cookie cutter from which we can get some cool tricks https://github.com/woltapp/wolt-python-package-cookiecutter It is actively maintained! In particular note that if offers automation regarding publishing to pypi 🙃

@drbenvincent
Copy link
Collaborator Author

Thanks. I don't know if this is a good idea, but the basic approach I'm trying to take is to learn by implementing everything manually, step by step. So it's slower and maybe more error prone, but hopefully I'll get a richer understanding in the end. At that point then automating with template repos or cookie cutter is probably the way forward. But yes, very wise to take inspiration from other repos!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
devops DevOps related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants