Skip to content

Fixed the exception twisted.web._newclient.RequestGenerationFailed#720

Merged
gkistanova merged 1 commit into
llvm:mainfrom
slydiman:fix-createStatus
May 17, 2026
Merged

Fixed the exception twisted.web._newclient.RequestGenerationFailed#720
gkistanova merged 1 commit into
llvm:mainfrom
slydiman:fix-createStatus

Conversation

@slydiman
Copy link
Copy Markdown
Contributor

@slydiman slydiman commented Jan 28, 2026

Added @defer.inlineCallbacks before def is_wrong_issue to make twisted happy.

And never use debug=True in GitHubStatusPush and inherited classes.
It will cause unexpected exceptions inside HTTP client service.

@slydiman slydiman force-pushed the fix-createStatus branch 4 times, most recently from 1ce3191 to cc369a2 Compare January 29, 2026 00:33
Added @defer.inlineCallbacks before def is_wrong_issue to make twisted happy.
Copy link
Copy Markdown
Contributor

@gkistanova gkistanova left a comment

Choose a reason for hiding this comment

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

LGTM.

@gkistanova gkistanova merged commit 452bbd4 into llvm:main May 17, 2026
4 checks passed
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