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

Fix prepare-release-pr script #13263

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix prepare-release-pr script #13263

wants to merge 2 commits into from

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

@nicoddemus nicoddemus added the backport 8.3.x apply to PRs at any point; backports the changes to the 8.3.x branch label Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 8.3.x apply to PRs at any point; backports the changes to the 8.3.x branch 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.

1 participant