Skip to content

Conversation

theshubhamp
Copy link
Contributor

Release creation fails with this error:

HTTP 422: Validation Failed (https://api.github.com/repos/RustPython/RustPython/releases)
body is too long (maximum is 125000 characters)
Error: Process completed with exit code 1.

Most likely because there's no previous release to diff of from.

Disabling, getting a green release & enabling back might fix this.

Release creation fails with this error:
```
HTTP 422: Validation Failed (https://api.github.com/repos/RustPython/RustPython/releases)
body is too long (maximum is 125000 characters)
Error: Process completed with exit code 1.
```

Most likely because there's no previous release to diff of from.

Disabling, getting a green release & enabling back might fix this.
@theshubhamp
Copy link
Contributor Author

@youknowone can you please check ? This & another follow up PR might fix the release notes character limit we are currently hitting.

Copy link
Member

@youknowone youknowone left a comment

Choose a reason for hiding this comment

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

Thank you!

@youknowone youknowone merged commit 1abaf87 into RustPython:main Dec 30, 2024
11 checks passed
@youknowone
Copy link
Member

@theshubhamp Releasing binary for every PR merge seems too much. Can this be a cron job like a weekly release? Also, to distinguish from periodic release from stable release, can this always-runnning releases be marked as pre-release?

@theshubhamp
Copy link
Contributor Author

@youknowone yes that makes sense. Had added on push main as a starting point, we should be able to swap that with a schedule.

I'll send a PR with weekly schedule and notes generation enabled. Monday GMT morning sounds fine ?

Do you see stable releases being manual short term ?

@theshubhamp
Copy link
Contributor Author

@youknowone handled in #5466. Could you check ?

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.

2 participants