Is release-drafter conditional on publishing to PyPI or TestPyPI?

[eyllanesc]

Reviewing the workflow release.yml, I see that if for some reason the publication fails in PyPI or TestPyPI then release-drafter is not executed:

cookiecutter-hypermodern-python/{{cookiecutter.project_name}}/.github/workflows/release.yml

Lines 58 to 79 in ce04c55

	- name: Publish package on PyPI
	if: steps.check-version.outputs.tag
	uses: pypa/[email protected]
	with:
	user: __token__
	password: {{ "${{ secrets.PYPI_TOKEN }}" }}
	- name: Publish package on TestPyPI
	if: "! steps.check-version.outputs.tag"
	uses: pypa/[email protected]
	with:
	user: __token__
	password: {{ "${{ secrets.TEST_PYPI_TOKEN }}" }}
	repository_url: https://test.pypi.org/legacy/
	- name: Publish the release notes
	uses: release-drafter/[email protected]
	with:
	publish: {{ "${{ steps.check-version.outputs.tag != '' }}" }}
	tag: {{ "${{ steps.check-version.outputs.tag }}" }}
	env:
	GITHUB_TOKEN: {{ "${{ secrets.GITHUB_TOKEN }}" }}

I think it should be executed anyway, and if so then a possible solution is:

      - name: Publish the release notes
        uses: release-drafter/[email protected]
        if: always() # <---
        with:
          publish: {{ "${{ steps.check-version.outputs.tag != '' }}" }}
          tag: {{ "${{ steps.check-version.outputs.tag }}" }}
        env:
          GITHUB_TOKEN: {{ "${{ secrets.GITHUB_TOKEN }}" }}

Another option would be to move release-drafter before PyPI and TestPyPI.

@cjolowicz What do you think?