Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EngSys] apply some code clean up for test and samples-dev #32917

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jeremymeng
Copy link
Member

This PR applies @mpodwysocki's clean-up code for the whole repo to reduce diffs
in snippets pull requests.

NO_CI

@github-actions github-actions bot added EngSys This issue is impacting the engineering system. Event Hubs Farmbeats Image Analysis KeyVault Maps Mgmt This issue is related to a management-plane library. Microsoft Playwright Testing Mixed Reality Monitor Monitor, Monitor Ingestion, Monitor Query OpenAI OpenTelemetryInstrumentation Purview Remote Rendering Schema Registry Search Service Bus Storage Storage Service (Queues, Blobs, Files) Synapse Tables test-utils Label for the issues related to the @azure/test-utils package WebPubSub labels Feb 6, 2025
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

////////////////////////////////////////////////////////
/////////////// Example for .byPage() ////////////////
////////////////////////////////////////////////////////
/// /////////////////////////////////////////////////////
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this should be intentional here.

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah that's the result of formatting. We should probably rewrite it to block style to prevent this from happening

This PR applies @mpodwysocki's clean-up code for the whole repo to reduce diffs
in snippets pull requests.

***NO_CI***
@jeremymeng jeremymeng force-pushed the engsys/samples-dev-cleanup branch from 1067dfb to 27f5f67 Compare February 10, 2025 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Untriaged
Status: Untriaged
Development

Successfully merging this pull request may close these issues.

3 participants