* Improve link checking for github links
Use locally cloned repos to check github links to avoid rate-limiting.
* Add default clone path to the link checker template
---------
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Support writing .env files from Test Resources
If a language repo opts into it *and* if a `test-resources.bicep` file exists and lints clean of writing secrets *and* if the `.env` file is gitignore'd, write a `.env` file next to `test-resources.bicep`.
* Resolve PR feedback
* Pass -Force for . hidden files on non-Windows
---------
Co-authored-by: Heath Stewart <heaths@microsoft.com>
* Add helper function for spec-gen-sdk pipeline
* Move logging script sourceing to global scope
---------
Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
* Properly handle error for case where no issue (pr) is found for specified commit
* Use logInfo in various places
* Change LogWarning to LogInfo
---------
Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
Given we are using the tenantId to determine to add the prefix or not we need to make sure we have it set before we check it. For the pipelines it is set at the script calling time so it was working there but for some local scenarios it is not set yet, so we need to do it a little later in the script.
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* when processing a PR diff, treat packages added via their triggeringPaths as directly changed (used to be indirect)
---------
Co-authored-by: Scott Beddall <scbedd@microsoft.com>
* Remove g++-8, not present in Ubuntu 22
* Update clang-11 job to Ubuntu 22
* Move live testing matrix to Ubuntu 22
* image.yml: Linux Next -> Linux
* "release" differentiator
* LINUXNEXT -> LINUX
* Deduplicate Ubuntu 22 in live matrix
* Trivial change to trigger CI
* Revert "Trivial change to trigger CI"
This reverts commit 72dae1d0cd90435cdad18391a993154a2b32c9e3.
* updating eng/common/testproxy/test-proxy-tool.yml and eng/common/testproxy/test-proxy-standalone-tool.yml to output a bunch of information about what they're actually running for their commands
---------
Co-authored-by: Scott Beddall <scbedd@microsoft.com>
Co-authored-by: Scott Beddall <45376673+scbedd@users.noreply.github.com>
Co-authored-by: Patrick Hallisey <hallipr@gmail.com>
* add a category to parse triggerPaths when there is no artifact present (this is the go use-case)
* add integration tests for Save-Package-Properties scenarios
* bugfixes around various situations in new PackageProps class
---------
Co-authored-by: Scott Beddall <scbedd@microsoft.com>
We need to use Write-Host for the custom logging commands to get them correctly out into stdout which is what they need to be interpreted correctly.
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* support accessing a different key for the value of the Package during batching in Create-PrJobMatrix
---------
Co-authored-by: Scott Beddall <scbedd@microsoft.com>
* Add logic for seting APIView Pull Request comment
* Update script to use source and head commish
* Make API Change Check a header
* Add Fetch URI to APIView comment
* Pass APIView host as a parameter
* Update APIVBiew comment header and add state to search api call
---------
Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
* identify and fix the issue with no results on a matrix that is PRBatched. The platform filters eliminate all the jobs, which is why it only turned up on the second windows leg
---------
Co-authored-by: Scott Beddall <scbedd@microsoft.com>
* update generate-matrix function ProcessImport to return a structured result, versus a tuple that is destructured immediately
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Scott Beddall <45376673+scbedd@users.noreply.github.com>
* first stab at a client tsp /tspconfig scripts
* space
* updates to clientTSP
* update for latest specs for KV
* gfd
* gfd
* hghf
* dfgdf
* jhg
* oopsie
* comments
* headers
* clang & param
* coverage
* more comments not gened
* spell
* more comments
* regen for testing
* gfd
* gdf
* tretre
* update to use tsp client
* vcxvx
* fdsfsd
* fds
* round 2
* api view
* dsa
* rte
* update to use code injection
* clang and comments
* gfd
* updates
* clang
* fsd
* spell
* comments
* some cleanup
* first test passing
* dsa
* second test
* startdeletesecret
* update secret ptoeprties
* recover secret
* cleanup clang
* docs
* update recordings and clean up tests
* ggfd
* ooops
* samples build
* gdf
* samples run
* perf tests
* CLANG
* fds
* CLAAAAAAAAANG
* update to latest version of the TSPs
* update to latest codegen
* gdfgdf
* replace method
* client and stuffs
* PR comments
* remove security from path
* clang
* qfe
* gdf
* update to models and to the lastest codegen
* forgot cmake file
* Update sdk/keyvault/azure-security-keyvault-secrets/samples/sample1-basic-operations/sample1_basic_operations.cpp
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
* test api view
* dfsfsd
* ccxcvx
* step1
* delete, and lor and paged
* all work
* works and serializers and test for them gone
* cleanup test files
* sample
* tests passing
* dfsfsd
* latest codegen
* after merge
* fsd
* gfd
* gfd
* ApiView
* move to _details namespace
* put back readme
* update readmes
* clang
* dfsd
* get rid of snippets, validation fails without explanation as to what the failure is
* update locations of methods tp cpp files
* format files
* cover
---------
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
Add the prefix to identify RGs that we are creating in our TME
tenant to identify them as potentially using local auth and violating
our safe secret standards.
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
