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

Change the TESTING_BUILD macro to make it clear it doesn't collide with a customer macro. #5334

Closed
ahsonkhan opened this issue Feb 9, 2024 · 0 comments · Fixed by #5390
Closed
Assignees
Labels
blocking-release Blocks release Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system. feature-request This issue requires a new behavior in the product in order be resolved. test-enhancement
Milestone

Comments

@ahsonkhan
Copy link
Contributor

Proposal is to rename the TESTING_BUILD compiler definition to _azure_TESTING_BUILD and use that repo-wide, so that it is unique to the C++ Azure SDK.

@RickWinter RickWinter added bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system. blocking-release Blocks release feature-request This issue requires a new behavior in the product in order be resolved. and removed bug This issue requires a change to an existing behavior in the product in order to be resolved. labels Feb 9, 2024
@ahsonkhan ahsonkhan added this to the 2024-03 milestone Mar 1, 2024
@github-actions github-actions bot locked and limited conversation to collaborators May 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocking-release Blocks release Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system. feature-request This issue requires a new behavior in the product in order be resolved. test-enhancement
Projects
None yet
2 participants