Skip to content

[dev-tool] Fix relative import of prettier options.#14560

Merged
witemple-msft merged 1 commit intoAzure:masterfrom
witemple-msft:dev-tool/fix-prettier-import
Mar 26, 2021
Merged

[dev-tool] Fix relative import of prettier options.#14560
witemple-msft merged 1 commit intoAzure:masterfrom
witemple-msft:dev-tool/fix-prettier-import

Conversation

@witemple-msft
Copy link
Member

The import path isn't quite correct and ts-to-js throws.

Copy link
Member

@deyaaeldeen deyaaeldeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a hunch my change would not work because require is dynamically called and indeed this is the case. Thanks for the fix! Please consider adding integration tests in this PR or in another.

@witemple-msft
Copy link
Member Author

Going to add a few tests for this in a different PR. Merging this to unblock samples publish.

@witemple-msft witemple-msft merged commit d6dd0d3 into Azure:master Mar 26, 2021
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this pull request Jul 29, 2021
Exposing reviewers property on accessReviewInstance model (Azure#14560)

* Cloning version 2021-07-01-preview from version 2021-03-01-preview

* Exposing reviewers properties on accessReviewInstance model

* Fixing SDK validation warnings

* Prettier check fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants