Skip to content

Repo sync for protected branch #5004

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

Merged
merged 4 commits into from
Mar 29, 2024

Conversation

learn-build-service-prod[bot]
Copy link
Contributor

Learn Build Service tried pushing to main, but failed due to branch protection rule in this branch. We create this pull request for manual merge. Please merge this pull request to complete this sync.
If you want to make push successfully without creating such pull requests in future, please follow working with branch protection rule to update your branch protection rule.

oldnewthing and others added 4 commits March 28, 2024 15:07
extern "C" does not remove decoration. It just changes to C decoration rules (instead of C++ rules). Perhaps the original author was deceived by the fact that on x64, the C decoration rules are "no decoration". But on x86, even C is decorated (by a leading underscore).
Copy link
Contributor

@learn-build-service-prod[bot] : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor Author

Learn Build status updates of commit 0d81cc5:

✅ Validation status: passed

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@learn-build-service-prod[bot] : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod learn-build-service-prod bot merged commit d11d6bb into main Mar 29, 2024
@learn-build-service-prod learn-build-service-prod bot deleted the main638473303524533460sync_temp branch March 29, 2024 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants