Skip to content

fix: remove redundant check#230

Merged
niemeyer merged 1 commit intocanonical:mainfrom
letFunny:remove-redundant-check
Jul 21, 2025
Merged

fix: remove redundant check#230
niemeyer merged 1 commit intocanonical:mainfrom
letFunny:remove-redundant-check

Conversation

@letFunny
Copy link
Collaborator

@letFunny letFunny commented Jun 24, 2025

  • Have you signed the CLA?

Removing a redundant check per the comment in #224 (comment).

@letFunny letFunny added Simple Nice for a quick look on a minute or two Polish Refactorings, etc labels Jun 24, 2025
@letFunny letFunny mentioned this pull request Jun 24, 2025
1 task
@github-actions
Copy link

Command Mean [s] Min [s] Max [s] Relative
BASE 8.771 ± 0.057 8.725 8.919 1.00
HEAD 8.800 ± 0.123 8.667 9.058 1.00 ± 0.02

Copy link
Contributor

@niemeyer niemeyer left a comment

Choose a reason for hiding this comment

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

Thanks! 👍

@niemeyer niemeyer merged commit eb48441 into canonical:main Jul 21, 2025
18 checks passed
@niemeyer
Copy link
Contributor

Uh.. this wasn't a "fix", btw.. too late.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Polish Refactorings, etc Simple Nice for a quick look on a minute or two

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants