Skip to content

Conversation

@svix-jplatte
Copy link

Summary

Fixes the unrelated clippy error that makes CI unhappy in #271.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@svix-jplatte
Copy link
Author

Oh, a CLA. Does this need to be signed even for a clearly non-copyrightable change (I didn't even delete these characters myself, that was clippy's --fix mode 😄)?

@slvrtrn slvrtrn closed this Aug 25, 2025
@slvrtrn
Copy link
Contributor

slvrtrn commented Aug 25, 2025

Closing this cause I pushed the fix without noticing this (sorry!).

Oh, a CLA. Does this need to be signed even for a clearly non-copyrightable change (I didn't even delete these characters myself, that was clippy's --fix mode 😄)?

That's a question to the legal team, but I'd prefer not to merge any PRs without it signed.

@svix-jplatte svix-jplatte deleted the clippy branch August 25, 2025 10:22
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.

4 participants