Sync eng/common directory with azure-sdk-tools for PR 995 #347
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Switch from Sleep to SpinWait
build cmocka libs path manually to VCPKG
* Update DPS to match other samples' structures * Fix potential memory leaks in get_operation_status.
…- ato[u|i][32|64], atod and the inverse [u|i][32|64]toa, dtoa. (Azure#885) * Add all permutations of number parsing and formatting APIs on span (atoi, u, d) with tests. * Fix linux and osx specific build errors and rename helper method. * Make sure whitespace or invalid characters aren't allowed when parsing numbers. * Fix OS specific compiler warnings, update format string, and fix tests. * Fix typo, cast to double, and ignore float-equal warnings in tests. * Update assert for double parsing that result in infinity, since it is compiler specific.
…ns, proper unescaping support, and hardened validation (Azure#871) * Initial commit of JSON parser changes. * Revert back to keeping az_json_token since json_pointer uses it. * Update parser impl and change token APIs to return AZ_ERROR_JSON_INVALID_STATE * Move is_text_equal to az_json_token and update signature of get_string. * Clean up headers and comments. * Update tests and fix up bugs. * Add invalid JSON payload tests that are exepcted to return unexpected char error code. * Fix up merge conflicts and add back some consts. * Cleanup included header files. * Remove az_json_pointer and related az_json_string helpers. * Add new line to parser file and disable unit testing. * Fixup skip children depth check and add unit tests. * Update paho twin sample and fixup uint8_t to char cast warning on OSX/Linux. * Fix up mismatch between docs and parameter names, and update local var name in iot hub sample. * Make sure all the fields of az_json_token are set in tests, including internal ones. * Fix iot hub sample be saving reported property value in a temporary uint. * Fix merge conflict from master - keyvault is gone. * Fix aad by adding the missing move_to_next token call. * Move span size check to a precondition and add an az_result which indicates that parsing is done. * Rename and shorten move_to_next_token API. * Update comments to mention token field is readonly and text equals is case sensitive. * Address PR feedback.
* Automatically update vcpkg on non-mac systems * Add version output * git pull origin master * git status on linux images? * git reset --hard HEAD * Don't set VCPKG_INSTALLATION_ROOT when it's already set
* [iot] Fixes to Azure IoT Hub ESP8266 sample - Renamed the directory so it can be opened directly on Arduino IDE - Added call to .loop() on pubsubclient - Added call to .setBufferSize() on pubsubclient, removed step to manually fix MQTT_MAX_PACKET_SIZE
…l JSON (Azure#897) * Update provisioning client parser to retain original logic for partial JSON. * Fix up the paho iot hub twin sample by adding a call to next_token.
* update version information * Add C to shipping languages * move verison up to dev.4 for testing * Remove ValidateSet * Enable generating artifacts, add replace-relative-links.yml * Add Update-Emojis.ps1 and pipeline syntax to update links and emojis * Use current directory when updating relative links * Replace only listed emojis * Fetch available emojis from the GitHub API * Revert "Remove ValidateSet" This reverts commit a19ba651041e87b75a8ebf1f2785a96c9d5cc815. * Revert "move verison up to dev.4 for testing" This reverts commit 19eebc1c2dfa7ff4708ab3f73a093ee153e50539. * Revert "Add C to shipping languages" This reverts commit 9b4bfe4b85f6af2be30493d43032dc3f7a49888b. * Revert "update version information" This reverts commit e6a1facbc0436665018ed290ca4b4e7d4172dbb4. * Use local emoji list for a repeatable build * Docs exclude recurisvely inside inc folder (cherry picked from commit 9379bcb3cef9495fca3ccdff34324cef28c30b49) * use html unicode symbol * Use HTML entities that render predictably in both GitHub and Doxygen * Remove Update-Emojis.ps1. We can use common html entities
* Add double support to JSON builder - az_json_builder_append_double_number * Add az_json_token_get methods for different integer sizes and doubles. * Make sure strings with negative values aren't allowed for unsigned parsing. * Fix tests for parsing infinity, and use _is_double_equal helper. * A really small number resulting in 0 is also compiler specific. * Address feedback, add comments for overflow/underflow cases. * Address PR feedback.
…zure#907) * Fix TODO to use int32 when parsing JSON in paho iot hub twin sample. * Address feedback, remove temporary local.
* adds (void) * fixes hex printout * checks punctuation
- out_, ref_, const - refactor to align with other samples - refactor to align thermostat and temp controller component functions - finalize logging output
Separates receiving the return code from a function call, and the actual check of the return code. Commenting and typo fixes.
* Use vcpkg cache correctly * vcpkg spelled more correctly * Only set VcpkgRef if we need it * Clean up parameters * Some more CI * quotes * Better dependency grouping * Set needed environment variables * Run as a string * command syntax * Update docs * . -> - * Rename vcpkg-ref.txt * Some review feedback * Upper case * OSVmImage is no longer necessary * Update docs to no longer mention vcpkg.yml, use vcpkg-ref.txt * Review feedback * Specify path directly... force and ErrorAction Ignore should already be fixing this * Only Remove-Item if Test-Path is * DevOps is complaining about another line * remove v2 from cache key
…tion_status to enum vaule (Azure#1250) * Changed `operation_status` in `az_iot_provisioning_client_register_response` from `az_span` to `az_iot_provisioning_client_operation_status` enum.
* az_iot_hub_client_twin_response_type enaumvaules rename to AZ_IOT_HUB_CLIENT_TWIN_RESPONSE_TYPE_XXXXX * Update CHANGELOG.md Co-authored-by: Ahson Khan <ahkha@microsoft.com> Co-authored-by: Ahson Khan <ahkha@microsoft.com>
adding licence for cmake modules
* Adding Notice for code coverage cmake module
9c08118 to
166b656
Compare
azure-sdk
pushed a commit
to azure-sdk/azure-sdk-for-android
that referenced
this pull request
Sep 16, 2020
* http request moved to internal * moving http request methods to internal headers * rename status code with underscore * make all private functions have _ as starting * remove unused include
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#995
See eng/common workflow