Skip to content

Comments

initial support for hyperx cloud flight wireless#211

Merged
Sapd merged 23 commits intoSapd:masterfrom
hede5562:master
May 24, 2022
Merged

initial support for hyperx cloud flight wireless#211
Sapd merged 23 commits intoSapd:masterfrom
hede5562:master

Conversation

@hede5562
Copy link
Contributor

@hede5562 hede5562 commented May 8, 2022

initial support for hyperx cloud flight wireless

There are several revisions of this headset but I do only have one. Maybe other revisions need further work but I didn't get any review on my fork.

@hede5562
Copy link
Contributor Author

fixed some problems with the code style... try again...

@hede5562
Copy link
Contributor Author

format-check fails were valid, but I do not see the point for the last whitespace failures as there are no longer any trailing whitespaces?

@Sapd Sapd merged commit 6c224e7 into Sapd:master May 24, 2022
@Sapd
Copy link
Owner

Sapd commented May 24, 2022

Thank you. Yeah there seems to be some problem at the CI whitespace check. Probably it does not check between the whole diff but some of the last commits.

@hede5562
Copy link
Contributor Author

Thank you for inclusion. Yes, the whitespace check seems to check those also for all intermediate patches. It seems without some kind of a rebase / squashing of those patches these kind of errors won't go away. Well, good to have a human reviewer here which can overrule automatic checks - seems way easier than the rebase/squash thing. ;-)

rpbaptist pushed a commit to rpbaptist/HeadsetControl that referenced this pull request May 31, 2024
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