Update working-main #5
Merged
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
* fix tool version * changelog --------- Co-authored-by: Catalina Peralta <[email protected]>
* Co-authored-by: catalinaperalta <[email protected]> Update package.json * Revert bundleDependency. * Restore bundleDependency * Revert change. * Restore bundleDependency * Replace local install with repo install. * Code review feedback.
* Add Button for Disabling CodeLines LazyLoadin * Add test for page options defaults
* adopt tsp-client command instead of pwsh script * update * ”update” * Revert "”update”" This reverts commit abe8f14.
The focus of these changes is to add the common service label for the "Compute - Fleet" resource provider.
* Co-authored-by: catalinaperalta <[email protected]> Update package.json * Revert change. * Update dependencies.
* tsp-client improvements * fix version range for typespec-autorest * changelog * changelog update --------- Co-authored-by: Catalina Peralta <[email protected]>
* revert 310645b * changelog update * changelog date --------- Co-authored-by: Catalina Peralta <[email protected]>
* update * update * update --------- Co-authored-by: Pan Shao <[email protected]>
* APIView Conversation Page * Show Comment Threads on Conversation Page * Ad test for comment thread * handle Save Comment from Conversation Page * Save, Edit and Delete form Conversation Page * Conversiation Page Fully Working * Adjust lastUpdated to last updated
…ipts to output limits and telemetry data (Azure#8830) * Add RestAPISpecsDocsRepos to branch cleanup. * Remove commented out yml, add pre-message to limit output and output core limit at the start of processing * updates for feedback
) * report error diagnostics and exit * release prep * update logic * wording * update changelog date --------- Co-authored-by: Catalina Peralta <[email protected]>
…e#8831) * we want the path that we write to to be defined by literalpath, otherwise files with [ or ] will break powershell when writing the file to disk * slight wording adjustment so the command can be invoked directly
Given we cannot create subfolders under .github\workflow folder and we have repos that want additional workflows then we have in the tools repo we need to setup a file pattern matching for the sync processing to only sync the ones we want to be common. For now just use the existing *event* pattern but we can move them to another pattern similar to *common* or *sync* in the future as we add more common workflows.
* upgrade openapi-to-typespec * update * add generated convert files * format run_commands.ts * update gitignore * release prep * newline * changelog * fix additional dir format * changelog --------- Co-authored-by: Pan Shao <[email protected]> Co-authored-by: Catalina Peralta <[email protected]>
* add gh action * review feedback * fix tests for latest update * add working dir * fix dir paths * sparse checkout * clean up code coverage * testing gh action - will revert * undo test commit --------- Co-authored-by: Catalina Peralta <[email protected]>
Adding the prefix "Workflow:" to the description of labels used in our issue workflow process.
* APIView code file token tsp and json schema
* Update deps. * Update tools/tsp-client/package.json Co-authored-by: Mike Harder <[email protected]> --------- Co-authored-by: Mike Harder <[email protected]>
Co-authored-by: Catalina Peralta <[email protected]>
* .NET API review parser changes to use new token schema
* Work on Subscribe Button * Subscribe Switch
…#8885) * Updated API review tokens to include two additional properties
* update package lock to deal with the component governance alert
* Add support for missing build processing * Add rust to list of monitored repos
* Add script for generating modular client * Update * wip * WIP * wip: remove sync call * Done: modular client SDK automation * WIP * WIP * minor fixes * Updated package.json * Improve log * Removed former files * Fixed typo * Added timeout for runCommand * Updated * Improved logs * Only run node test * Removed len == 1 array * Updated package version * Fixed template not found bug * Update build script * Updated package.json * Fixed sdk type bug * Fixed no throw issue * Fix * Update package.json * Update lock file * Update * a * a * Revert "a" This reverts commit 8d3535c. * Revert "a" This reverts commit fcb860a. * debug * Revert "debug" This reverts commit f7d0eb9. * Remove dup code * Fix merge conflict * Update * Normalize path * Update apiViewArtifact declaration * Update path * Workaround to pass pipeline * Update mgitignore * Update api view json * Remove * Update logger * . * Remove useless log * . * Comment * Add debug info * . * Silent log * Debug * Copy api view to somewhere else to avoid clean up by npm script * . * Cleanup * Update rush script * Update rush script * Add missing package * chore: Remove shx dependency and update npm packages * Update CI creator * Add log * Update * Update MC check * Update log level * Support emitter 0.32 in api version extraction * Change temp folder location * Add log * Add debug info * Fix package info * Fix * Improve logging * Update logging * . * Update logging * Update log * RLC update * Simplify * Add default package name * Fix runCommand * Fix parameters finding * Fix run command issue * Fix runCommand issue * Improve logging * Add error keyword * WIP on wanl/generate-mlc-in-pipeline * . * WIP * Test sdk generation path * Fix original package version restore * . * cleanup * Update version * Update package.json
* Add CI to verify APIView token schema
Merge azure-sdk-tools main into main
* Update PR triage rule for PRs created by BOTs * fix mistype of label
* Allow uploads of dll files * Update File Upload Labels
…ow in keyvault admin on debug builds (Azure#8749) * Added vcpkg baseline; Updates for clang 18; Worked around stack overflow in keyvault admin on debug builds * Removed debug code * Added a comment explaining why the stack needs to be bigger * Added error on using namespace in header because it can easily lead to duplicate errors
…zure#8902) * Add descriptive error when variable groups are not added to pipeline * Add emoji to deploy step title to make it easier to find * Move yaml json checking into powershell script * Fix base sub config empty/string checking
* don't log errors that we end up raising #22557
Added new rule for this issue to the pylint_guidelines_checker file.
Added corresponding tests to the test_pylint_custom_plugins file
* test_pylint_custom_plugins - added more tests
Added two more tests to check for multiple exception blocks
* added visit_functiondef() and visit_classdef() functions to checker class
* Changed 'msg' to 'msgs' and removed un-needed functions
* Made matches more specific
Changed 'warning' to '.warning' etc. Help avoid any false positives e.g. 'print("error")' will now not trigger Pylint
* Update README to include rule
* Update checker to only include 'warning' and 'error' logging levels
Also added another test to test_pylint_custom_plugins.py for 'warning' level.
* Update to account for branches within exception blocks
Also rename checker to "do-not-log-raised-errors"
* Make logging levels case insensitive
Added '.lower()' to string being parsed for logging.
* Update tools/pylint-extensions/azure-pylint-guidelines-checker/pylint_guidelines_checker.py
Co-authored-by: Anna Tisch <[email protected]>
* Added checking for raises in 'for' branches
Also added relevant tests
* Altered 'check_for_raise' helper function
Refactored function and removed 'isInstance Astroid.For' and associated tests
---------
Co-authored-by: Anna Tisch <[email protected]>
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.
No description provided.