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

refactor!: poetry, github action overhaul, pyenv / asdf versions #137

Closed
wants to merge 7 commits into from

Conversation

tony
Copy link
Member

@tony tony commented Oct 15, 2022

Fixes #136

@tony tony force-pushed the poetry-py.test branch 3 times, most recently from f570809 to 1dc921f Compare October 15, 2022 01:24
Comment on lines +74 to +80
- name: Publish package
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags')
uses: pypa/gh-action-pypi-publish@release/v1
with:
user: __token__
password: ${{ secrets.PYPI_API_TOKEN }}
skip_existing: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be done here but in the release.yml file instead. Please don't push to PyPI directly, too.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's true ☝️ And I know why that'd be. This is copied from my own project's workflows (it wasn't meant to be added)

Copy link
Member

@jezdez jezdez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have to be honest, I'd rather switch to hatch than poetry to be honest, as it's probably better supported via the PyPA. I like seeing the move from setup.py to pyproject.toml though!

@@ -0,0 +1,24 @@
session_name: django-robots
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That seems not needed here.

@@ -0,0 +1,2 @@
poetry 1.2.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't needed for development should be kept on your machine.

@tony
Copy link
Member Author

tony commented Oct 16, 2022

@jezdez I meant to close this PR. This was just experimenting ^

@tony
Copy link
Member Author

tony commented Oct 16, 2022

I'd rather switch to hatch than poetry to be honest, as it's probably better supported via the PyPA.

hatch, that's totally new to me!

I like seeing the move from setup.py to pyproject.toml though!

I'm happy to contribute that in a separate PR (and even try hatch out)

That said - the bigger issue I'm concerned about is #124.

@jezdez What are your thoughts on this project here and now? Am I doing more harm than good? Do you want to pilot it again? (I notice this specific django package is your creation, on atop of your being apart of jazzband). Can / should I email you?

@tony
Copy link
Member Author

tony commented Oct 16, 2022

For now I will close this - per #137 (comment)

@tony
Copy link
Member Author

tony commented Oct 18, 2022

@jezdez

Where I took this:

  1. re: hatch, pipenv, poetry, etc: Package management help#313

  2. Best practices: Best practices / standards: Workgroup + sprints? help#314

    in the above I also floated the idea of:

    • a "flagship" project that can champion the best practices for other projects to imitate
    • potential sprints to update best practices by hand (e.g. setup.py -> pyproject.toml across all projects)

@ofek
Copy link

ofek commented Oct 19, 2022

potential sprints to update best practices by hand (e.g. setup.py -> pyproject.toml across all projects)

fyi Hatch can pretty robustly automate that https://hatch.pypa.io/latest/intro/#existing-project

@ofek
Copy link

ofek commented Oct 24, 2022

I did one 🙂 django-commons/django-debug-toolbar#1690

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.

Test infrastructure, packaging, etc. needs overhaul
3 participants