Skip to content

Conversation

@timterle
Copy link

No description provided.

ozangunalp
ozangunalp previously approved these changes Oct 31, 2025
Copy link
Collaborator

@ozangunalp ozangunalp left a comment

Choose a reason for hiding this comment

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

Looks good to me. However, no matter what I do, I couldn't reproduce the negative credits.

@ozangunalp
Copy link
Collaborator

@timterle ok for me. Could you sign your commits for me to merge the change ?

Thank you!

@timterle
Copy link
Author

timterle commented Nov 3, 2025

I managed to implement a test that triggers "invalid request number", but it runs for 6 seconds. I don't know if that is acceptable?

@timterle
Copy link
Author

timterle commented Nov 3, 2025

@ozangunalp I tried to rebase and sign the commits, does it look ok for you? I don't see "Verified", maybe it's because I managed to commit using my git user at work, but making a pull request using my personal account.

@ozangunalp
Copy link
Collaborator

@timterle I think you signed-off but did not signed your commits. Here is the GH guide for it https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

@timterle timterle force-pushed the main branch 4 times, most recently from 4cb7651 to d386272 Compare November 3, 2025 13:30
@ozangunalp
Copy link
Collaborator

duplicate of #3213

@ozangunalp ozangunalp closed this Nov 4, 2025
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.

3 participants