Conversation
Used in the release pipeline.
upload-to-storage, which was previously converted to JS, had linting issues. Also converted the utils module packaged with the pipeline scripts to TypeScript.
This was breaking due to other dependency changes, but shouldn't be needed
Added markdown linting
Also suppresses warnings in `npm run eslint`
* ARM API versions can now be specified per-resource Added an API version for listing compute SKUs * Update vm-size.ts and vm-size.service.ts with new resourceSku API data * Check for response.value always 0 indexed * Clean up console logs * Fix url for arm response call * Update vmSize test suite for new response data * Update tests for bad response data and extra capability response data * Clean up console logs * Fix linting issues * Fix vm sizes table not showing up Co-authored-by: David Watrous <[email protected]>
…2298) * Fix vrayrt error (application license id not found issue) * Merged V-Ray and V-Ray RT into the same option * Minor test fix Co-authored-by: David Watrous <[email protected]> Co-authored-by: Shiran Pasternak <[email protected]>
…nvisible main-window (#2296)
Updated email addresses
…images, and pay-for-use licensing (#2307) * Add deprecation banner for gallery * Color style changes to meet accessibility requirements * Add license picker deprecation warning for graphics and rendering * Add deprecation warnings for cloud service pools
* Updated changelog and version. * Updated changelog and version. * Update changelog * Fix linting for changelog Fix linting * Uncomment updating changelog Removed temporarily to generate a new changelog for 2.9.0 * Override markdown linting rules MD001 and MD025 Co-authored-by: rechen <[email protected]>
cRui861
requested review from
bgklein,
dpwatrous,
gingi,
paterasMSFT and
zfengms
as code owners
Codecov Report
@@ Coverage Diff @@
## stable #2319 +/- ##
=========================================
Coverage ? 65.25%
=========================================
Files ? 904
Lines ? 25763
Branches ? 5115
=========================================
Hits ? 16811
Misses ? 8952
Partials ? 0
Continue to review full report at Codecov.
|
dpwatrous
approved these changes
Apr 29, 2021
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Re-opening another PR because the previous codecov report was broken.
Adding onto this 2.9.0 release (#2288) because it wasn't deployed.