Skip to content

Release 1.7.4 - #104

Merged
nicoddemus merged 13 commits into
masterfrom
release-1.7.0
Aug 8, 2026
Merged

Release 1.7.4#104
nicoddemus merged 13 commits into
masterfrom
release-1.7.0

Conversation

@nicoddemus

Copy link
Copy Markdown
Member

No description provided.

Clarify to push the branch to upstream and mention the expected tag format.
* Build package once with `hynek/build-and-inspect-python-package` and test in multiple versions.
* Use `hynek/build-and-inspect-python-package` during deploy.
Cannot use attestations together with a password; we might use attestations in a follow up.
Seems like username and password are not supported anymore.
@nicoddemus

Copy link
Copy Markdown
Member Author

@RonnyPfannschmidt I don't have admin access to PyPI, could you please setup trusted publishing, using the deploy workflow? Thanks

@nicoddemus nicoddemus changed the title Release 1.7.0 Release 1.7.4 Aug 5, 2026
@nicoddemus

Copy link
Copy Markdown
Member Author

@RonnyPfannschmidt did you setup trusted publishing?

@RonnyPfannschmidt

Copy link
Copy Markdown
Member

Not that I remember

I can check later

@nicoddemus

Copy link
Copy Markdown
Member Author

Thanks, please see #104 (comment).

@RonnyPfannschmidt

Copy link
Copy Markdown
Member

configured

@nicoddemus

Copy link
Copy Markdown
Member Author

Hmm failing with:

  Token request failed: the server refused the request for the following reasons:
  
  * `invalid-publisher`: valid token, but no corresponding publisher (Publisher with matching claims was not found)
  
  This generally indicates a trusted publisher configuration error, but could
  also indicate an internal error on GitHub or PyPI's part.
  
  
  The claims rendered below are **for debugging purposes only**. You should **not**
  use them to configure a trusted publisher unless they already match your expectations.
  
  If a claim is not present in the claim set, then it is rendered as `MISSING`.
  
  * `sub`: `repo:pytest-dev/pytest-forked:ref:refs/tags/v1.7.5`
  * `repository`: `pytest-dev/pytest-forked`
  * `repository_owner`: `pytest-dev`
  * `repository_owner_id`: `8897583`
  * `workflow_ref`: `pytest-dev/pytest-forked/.github/workflows/deploy.yml@refs/tags/v1.7.5`
  * `job_workflow_ref`: `pytest-dev/pytest-forked/.github/workflows/deploy.yml@refs/tags/v1.7.5`
  * `ref`: `refs/tags/v1.7.5`
  * `environment`: `MISSING`

Would you mind adding me as maintainer on PyPI? Then I can debug the configuration myself.

@RonnyPfannschmidt

Copy link
Copy Markdown
Member

added core as owners

@RonnyPfannschmidt

Copy link
Copy Markdown
Member

I had a typo in the repo name

@nicoddemus
nicoddemus merged commit 220885d into master Aug 8, 2026
21 of 22 checks passed
@nicoddemus
nicoddemus deleted the release-1.7.0 branch August 8, 2026 12:05
@webknjaz

webknjaz commented Aug 8, 2026

Copy link
Copy Markdown
Member

environment: MISSING

@nicoddemus hey, I always recommend having the environment set to pypi (and then adding an additional approval protection in the repo settings for it, FYI).

@nicoddemus

Copy link
Copy Markdown
Member Author

True, done: #105

pull Bot pushed a commit to sysfce2/python-pytest-forked that referenced this pull request Aug 9, 2026
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.

3 participants