Skip to content

Fix prepare-release-pr script #13262

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

Closed
wants to merge 1 commit into from
Closed

Fix prepare-release-pr script #13262

wants to merge 1 commit 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.


Note: decided to push to 8.3.x first so I can test it right away -- I will port back to main afterwards.

@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 e59f327 to fdd82b4 Compare March 2, 2025 12:31
@nicoddemus nicoddemus closed this Mar 2, 2025
@nicoddemus
Copy link
Member Author

I will reopen this from main -- the origin of the branch is independent of which branch we want to make the release from anyway.

@nicoddemus nicoddemus deleted the fix-prepare-release branch March 2, 2025 12:37
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.

1 participant