Skip to content

Integrate from main#36202

Merged
ankushbindlish2 merged 17 commits intorelease-storage-Microsoft.FileShares-preview/2025-06-01-previewfrom
main
Jul 25, 2025
Merged

Integrate from main#36202
ankushbindlish2 merged 17 commits intorelease-storage-Microsoft.FileShares-preview/2025-06-01-previewfrom
main

Conversation

@ankushbindlish2
Copy link
Member

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

Click here to open a PR for only SDK configuration.

bidisha-c and others added 17 commits July 24, 2025 10:54
* [Language] Remove old preview API versions

* Remove 2023-04-15-preview from authoring

* update the api version

* tsv compile

---------

Co-authored-by: Bidisha Chakraborty <bidishac@microsoft.com>
* Add package.json

* Enable eng/tools in JS

* First cut at direct port

* Wire up command, move some functions into changed-files.js

* Move, pipeline, refactor

* Trivial test change to a spec file

* Usage, logging, add pathExists to changed-files.js, types in doc-preview.js, coverage for doc-preview.js

* Prettier

* SkipCheckoutNone

* Git token auth

* --

* ../

* Pool

* fetchDepth: 2

* Get-ChildItem

* More logging

* Directory

* BaseRepoOwner

* Cleanup

* 02

* Comment

* Test a change that should break the docs build

* Queue docs build

* 02

* Tab

* Add wait and result output

* Log files

* Direct invocation with valid path

* Output buildstart.json

* Fixes

* Remove BOM

* PR Number

* Test changes that won't break the build

* encodeURIComponent

* -pwsh

* Revert BatchService.json testing

* Test from another PR

* Test timeouts

* Test timeout in script

* >

* Remove test exception

* Show token scope

* Set status on PR

* git rev-parse HEAD

* SourceCommitId

* -Azure/

* Set check state in DevOps pipeline

* set-pr-check.yml

* -description

* Can't enforce values for macro (defined at runtime) syntax at template expansion time

* .mjs -> .js

* .js, other documentation

* vso

* displayName

* Add package.json to eng/scripts, wire up to github-test.yaml (might need a rename)

* sparse checkout eng/scripts

* eng/scripts/package-lock.json

* package.json

* npm i

* prettier -> format

* package-lock.json

* --output

* Path

* Revert test changes in specification/

* Revert "Test from another PR"

This reverts commit 04aeecc.

* Architecture feedback

* Paths

* Docs preview

* pscore

* `

* copy

* $buildStart

* Test: timeout

* (

* Remove test timeout

* Test orchestration build failure

* Revert contrived orchestration failure

* Contrive a docs build failure

* Test success case

* Revert spec change

* Add support for filtering quickstart templates

* Use aka.ms link for docs support teams channel

* Sparse checkout might work but that can be investigated elsewhere

* Improve param validation

* Test api-doc-preview param validation in practice

* Revert github-test.yaml

* Revert .gitignore

* Remove unnecessary files

* More status update messages

* Revert "Test api-doc-preview param validation in practice"

This reverts commit 929d33c.

* Fix package.json/package-lock.json

* package-lock.json

* npm i

* Format

* Apply suggestions from code review

Co-authored-by: Mike Harder <mharder@microsoft.com>

* inline, fix ts-check errors

* Use getChangedFilesStatuses to exclude deleted files instead of querying the filesystem directly

* Comments

* Named typedefs, more exceptions, expand "key" to a couple variables

* Adjust PR triggers

* Remove fs/promises/access

* Add WorkingDirectory to npm-install.yml, use shebang invocation

* Remove infinite loop condition

* pathExists

* .js

* (

* Only run docs build if there are changes to push

* DisplayName

* Final TODOs

* Format

* Update eng/pipelines/swagger-api-doc-preview.yml

Co-authored-by: Mike Harder <mharder@microsoft.com>

---------

Co-authored-by: Mike Harder <mharder@microsoft.com>
…ppear in the docs. (#35646)

* ordered the enum tables how they should appear in the docs. Updated enum descriptions.

* ordered the enum tables how they should appear in the docs. Updated enum descriptions.

* minor updates

* removed blank line.

* Removed word 'inhalable'

---------

Co-authored-by: Faraz Siddiqui <124096851+FarazGIS@users.noreply.github.com>
Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
* Moving specs from private to public

The files are the same as the one in https://github.com/Azure/azure-rest-api-specs-pr/pull/20074/files#diff-4764111bf455581c775da5c4711ccd11e820292fc9c84957532fc5d278b3ea2d

* Added missing files

* Add only version 2024-01-01 preview

* Remove old versions

* add back the old versions

* Removing old versions

* Added older version of the API

* Added older version of API to Readme

* Update version.tsp

* remove 2022 versions

* Delete versions prior 2024-06-01-preview

* remove status 200 of delete operations

* remove status 200 from openapi.json

* remove status 200 from example files

* remove status 200 from EnclaveEndpoints_Delete.json

* remove the status 200 when compiling

* change ArmResourcePatchAsync to ArnCustomPatch

* compile file with ArmCustomPatch

* rollback to ArmResourcePatch

* added version 2024-12-01

* remove status 200 from ver 2024-12-01

* change ARMResourcePatch for ARMResourcePatch

* add @parameterVisibility to all ocurrences of ArmCustomPatchAsync

* minor fix on ArmCustomPatchAsync

* compile the model

* added location to Community Update

* location to community_update

* Update Community_Update.json

* added location to CommunityEndpoints

* Added location on all update example files

* added location to virtualEnclave_update

* made rulecollection optional

* change propertires to optional

* change properties to optional

* Added suppression

* add suppression

* rollback suppressions

* made properties optional

* update openapi file

* Update for tsp update

* Update formatting

* Use latest version v6 of types.json

* Use latest version v6 of types.json

* Update patch model

* Update patch model

* Update patch model

* Update patch model

* Update patch model

* Update patch model. Revert location addition in examples

* Update patch model

* Update patch model

* Update patch model

* Update patch model. Fix location in examples.

* Update patch model. Fix location in examples.

* Format update

* Format update. Use Azure.Core.azureLocation for location def in mrg

* Fix visibility for mrg location

* Remove 2024_10 and 2024_11

* Remove 2024_10 and 2024_11 examples

* Remove 2024_10 and 2024_11 from readme.md

* Remove 2024_10 and 2024_11 from resource-manager

* Use Update<T> instead of separately defining the patchable props

* Use Update<T> on patchResources

* Revert optional/required params to match ve-common and hope the Update<T> model handles it correctly.

* Re-Enable PATCH optionality.

* Add types for ARM resources

* Fix examples

* Add suppressions for AvoidAnonymousTypes

* Fix examples for type updates. Add uuid type to uuid fields.

* Fix examples for type updates.

* Remove uuid type

* Remove uuid type

* Prettier

* Prettier

* Update suppressions

* Move routes to routes.tsp

* Feedback from engineering -- remove Endpoint, InternalConnection, ExternalConnection models and interfaces. Update Community to include _ as part of name validation.

* Fix examples

* Remove examples for deprecated resources

* Remove examples for deprecated resources

* Remove examples for deprecated resources

---------

Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: Trevor Goodyear <tgoodyear@microsoft.com>
Co-authored-by: Trevor Goodyear <tgoodyear@users.noreply.github.com>
…on.md to singleapi package format (#36026)

* Initial plan

* Convert applicationinsights readme.python.md to singleapi package format

Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>

* Update readme.java.md

* Update readme.java.md

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Weidong Xu <weidxu@microsoft.com>
* Install .NET from tools repo global.json

* Set workingDirectory to tools root

* use exact same install steps as tools repo

* revert formatting

* formatting

---------

Co-authored-by: Mike Harder <mharder@microsoft.com>
…6172)

All files and directories under specification/recoveryservices/Recoveryservices.Management were renamed to specification/recoveryservices/RecoveryServices.Management to standardize casing and improve consistency.
…6169)

All files and folders under specification/powerbidedicated/Powerbidedicated.Management were renamed to specification/powerbidedicated/PowerBIDedicated.Management to correct casing and improve consistency.
Renamed 'Notificationhubs.Management' to 'NotificationHubs.Management' across all specification files and examples for consistency. Also updated the service name in tspconfig.yaml to 'NotificationHubs'.

Co-authored-by: Dapeng Zhang <dapzhang@microsoft.com>
@ankushbindlish2 ankushbindlish2 requested a review from MattGal as a code owner July 25, 2025 09:59
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Jul 25, 2025

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ Your PR has breaking changes in the generated SDK for Go (label: BreakingChange-Go-Sdk). Refer to step 3 in the PR workflow diagram.
  • ❌ Your PR has breaking changes in the generated SDK for JavaScript (label: BreakingChange-JavaScript-Sdk). Refer to step 3 in the PR workflow diagram.
  • ❌ Your PR has breaking changes in the generated SDK for Python (label: BreakingChange-Python-Sdk). Refer to step 3 in the PR workflow diagram.

@openapi-pipeline-app
Copy link

PR validation pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@ankushbindlish2 ankushbindlish2 merged commit 9d5a6e9 into release-storage-Microsoft.FileShares-preview/2025-06-01-preview Jul 25, 2025
4669 of 4823 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.