Update breaking change tool and oav

[vladbarosan]

No description provided.

[AutorestCI]

Automation for azure-sdk-for-node

Unable to detect any generation context from this PR.

[AutorestCI]

Automation for azure-libraries-for-java

Unable to detect any generation context from this PR.

[AutorestCI]

Automation for azure-sdk-for-python

Unable to detect any generation context from this PR.

[AutorestCI]

Automation for azure-sdk-for-go

Unable to detect any generation context from this PR.

[azuresdkciprbot]

AutoRest linter results for SDK Related Validation Errors/Warnings

These errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns.

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

[azuresdkciprbot]

AutoRest linter results for ARM Related Validation Errors/Warnings

These errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns.

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

[hovsepm]

@vladbarosan why do we have "package-lock.json" file in the repo? this should be just a local temporary file that is created after npm install command.

[hovsepm]

@amarzavery since you were the original creator of this file :) Could you explain why do we need it in the repo?

[vladbarosan]

@hovsepm not sure whats the history of it but since npm 5.1 package.json indeed trumps the lock file when using npm install. Unless we want to move to npm ci which was released in the latest npm version we can remove the lock file. ( npm ci uses the lock file)

[vladbarosan]

synced with @amarzavery and will be staying with npm install for the time being.

[azuresdkciprbot]

AutoRest linter results for ARM Related Validation Errors/Warnings

These errors are reported by the ARM team's validation tools, reachout to ARM RP API Review directly for any questions or concerns.

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

[azuresdkciprbot]

AutoRest linter results for SDK Related Validation Errors/Warnings

These errors are reported by the SDK team's validation tools, reachout to ADX Swagger Reviewers directly for any questions or concerns.

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

[hovsepm]

@sarangan12 could you take a look and LMK if you are ok with merging this PR?