* Added more logging to API view create
* Updated logs with arch board rescue email address
* Fix as per review comments
* Update logging to show API URL
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
* Deleting unused files and dir from eng dir
* Copying eng content from master commit: 101fcb3653
* Adding ci.yml for core modules
* Applying junit5 plugin to enable running tests from terminal
* Updating build.gradle to produce jacco report
* Updating gradle to v6.4.1
* Excluding jdk internal classes from jacoco
* jacoco: attempting to exclude jdk internal class by path
* Fixing NPE in jacoco config
* Attempting to use java8 image
* Using ubuntu 18
* Adding test package for credential and logging module
- Remove BuildType parameter as we can default it from package properties
- Stop passing BuildType and GroupId and instead defaul them from package properties
- Enable StrictMode to help identify potential errors
- Start passing sdktype and isnewsdk properties to devops script
- Sync latest changes with devops work item to fix a couple bugs
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Fix incompatibility between String and CommunicationUserIdentifier when serialization
* version number update on root folder
* Remove the test app
* Fix link in README.md
Add one example to README.md
* Update CHANGELOG.md
* Update version in both packages' CHANGELOG.md and build.gradle
* Update common's version in common ReadMe and root ReadMe
Co-authored-by: Marc Ma <marcma@microsoft.com>
* Fix incompatibility between String and CommunicationUserIdentifier when serialization
* version number update on root folder
* Remove the test app
* Fix link in README.md
Add one example to README.md
* Update CHANGELOG.md
Co-authored-by: Marc Ma <marcma@microsoft.com>
* Add common tools for spell check
* Updates from further iteration
* Review feedback: Add check for git as well
* Review feedback * Use common approach to resolving base branch name * Eliminate default base branch "master" as this will be changed later, providing no default and using a mandatory parameter means a dev must provide the value * Check for existence of $CspellConfigPath * No need to Set-Location if run from the root of the repo (most common scenario) * -join
* Review feedback: Rename TargetRef -> TargetBranch, add SourceBranch, Add reference to spelling docs, exit 0, Rename to check-spelling-in-changed-files.ps1,
* Review feedback: Remove ValidateNotNullOrEmpty (we do more definitive validation farther down that will also catch these errors), Update comments and script name
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* azure-core-rest minApi-15: baselining azure-core-rest to Api Level 15.
* Given rest and serde-jackson modules are L15 now, removing the temporary overrideLibrary for them from HttpClient impls
* Updating azure-core-serde module's build.gradle be to explict about minSdkVersion:15
* azure-core--http-[httpclient]* minApi: baselining azure-core-http-httpurlconnection to Api Level 15 and azure-core-http-okhttp to api Level 21 (21 is enforced by okhttp).