Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport #76153 #76154

Closed
wants to merge 1 commit into from
Closed

Backport #76153 #76154

wants to merge 1 commit into from

Conversation

GuardianDll
Copy link
Member

Summary

None

Purpose of change

Backport #76153

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Sep 2, 2024
@Maleclypse
Copy link
Member

@akrieger would this be a risky Backport?

@GuardianDll
Copy link
Member Author

huh, what's with this error in basic build test?
image

@akrieger
Copy link
Member

akrieger commented Sep 2, 2024

Well I wouldn't recommend it until I fix the tests on the original. Which tbh I don't understand why they are failing but there's a nonzero chance I just delete them because I am dubious about the assertion being useful.

@GuardianDll GuardianDll marked this pull request as draft September 3, 2024 06:17
@GuardianDll
Copy link
Member Author

original PR suddenly transformed to the point i don't feel myself good enough to backport it on my own

@GuardianDll GuardianDll closed this Sep 6, 2024
@GuardianDll GuardianDll deleted the backport_optimizations branch September 6, 2024 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants