Skip to content

Conversation

@nicoddemus
Copy link
Member

The script is currently failing with:

remote: Support for password authentication was removed on August 13, 2021.
remote: Please see https://docs.github.com/get-started/getting-started-with-git/about-remote-repositories#cloning-with-https-urls for information on currently recommended modes of authentication.
fatal: Authentication failed for 'https://github.com/pytest-dev/pytest.git/'

Decided to remove the usage of github3 and use the gh command-line tool directly, which simplifies the script, integrates nicely with GH, and enables to run it locally seamlessly.

@nicoddemus nicoddemus added the skip news used on prs to opt out of the changelog requirement label Mar 2, 2025
The script is [currently failing with](https://github.com/pytest-dev/pytest/actions/runs/13615071695/job/38057071681):

```
remote: Support for password authentication was removed on August 13, 2021.
remote: Please see https://docs.github.com/get-started/getting-started-with-git/about-remote-repositories#cloning-with-https-urls for information on currently recommended modes of authentication.
fatal: Authentication failed for 'https://github.com/pytest-dev/pytest.git/'
```

Decided to remove the usage of `github3` and use the `gh` command-line tool directly, which simplifies the script, integrates nicely with GH, and enables to run it locally seamlessly.
@nicoddemus nicoddemus force-pushed the fix-prepare-release branch from db85ff2 to 79f0733 Compare March 2, 2025 12:44
@nicoddemus
Copy link
Member Author

🎉 #13264

Copy link
Member

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Choose a reason for hiding this comment

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

the way this script develops - i wonder if it wouldn't make sense to just turn it into a workflow later on

@nicoddemus nicoddemus merged commit ec87085 into main Mar 13, 2025
28 checks passed
@nicoddemus nicoddemus deleted the fix-prepare-release branch March 13, 2025 14:03
@patchback
Copy link

patchback bot commented Mar 13, 2025

Backport to 8.3.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.3.x/ec87085035996b718af8dc680caa461f0e49b854/pr-13263

Backported as #13295

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 13, 2025
Fix prepare-release-pr script

(cherry picked from commit ec87085)
nicoddemus added a commit that referenced this pull request Mar 13, 2025
Fix prepare-release-pr script

(cherry picked from commit ec87085)

Co-authored-by: Bruno Oliveira <nicoddemus@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news used on prs to opt out of the changelog requirement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants