-
-
Notifications
You must be signed in to change notification settings - Fork 100
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
Conversation
f570809
to
1dc921f
Compare
- 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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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)
There was a problem hiding this 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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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.
@jezdez I meant to close this PR. This was just experimenting ^ |
hatch, that's totally new to me!
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? |
For now I will close this - per #137 (comment) |
Where I took this:
|
fyi Hatch can pretty robustly automate that https://hatch.pypa.io/latest/intro/#existing-project |
I did one 🙂 django-commons/django-debug-toolbar#1690 |
Fixes #136