Skip to content

http-client-js: improved default output formatting #9270

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.

Sign up for GitHub

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

Jump to bottom
Open
johnpyp wants to merge 1 commit into
base: main
Choose a base branch
from
Open

http-client-js: improved default output formatting #9270

johnpyp wants to merge 1 commit into from

Conversation

@johnpyp
Copy link

@johnpyp johnpyp commented Dec 28, 2025

This PR:

  • adds support for disabling formatting in scenario test files
  • Fixes some simple easy formatting wins for http-client-js clients, like avoiding double semicolons and making sure newlines are consistent. Just from my personal observations playing around with the client. There's definitely more, but these are some of the most visible when doing go-to-definition and such into the generated code.
    • adds a decent_formatting.md scenario to test for this kind of thing when they're noticed.

@johnpyp
http-client-js: improve default output formatting
d648b7f 
some basic, easy things like removing double-semicolons, consistent
newlines, consistent semicolon formatting for class members.

also added a `decent_formatting.md` scenario for this
@johnpyp johnpyp force-pushed the feat/http-client-jsbetter-formatting branch from 75733c1 to d648b7f Compare December 28, 2025 23:05
@johnpyp
Copy link
Author

johnpyp commented Dec 28, 2025

@microsoft-github-policy-service agree

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

Reviewers

@joheredi joheredi Awaiting requested review from joheredi joheredi is a code owner

@timotheeguerin timotheeguerin Awaiting requested review from timotheeguerin timotheeguerin is a code owner

@timovv timovv Awaiting requested review from timovv timovv is a code owner

@qiaozha qiaozha Awaiting requested review from qiaozha qiaozha is a code owner

@MaryGao MaryGao Awaiting requested review from MaryGao MaryGao is a code owner

@xirzec xirzec Awaiting requested review from xirzec xirzec is a code owner

@bterlson bterlson Awaiting requested review from bterlson bterlson is a code owner

@markcowl markcowl Awaiting requested review from markcowl markcowl is a code owner

@witemple-msft witemple-msft Awaiting requested review from witemple-msft witemple-msft is a code owner

@chrisradek chrisradek Awaiting requested review from chrisradek chrisradek is a code owner

@AlitzelMendez AlitzelMendez Awaiting requested review from AlitzelMendez AlitzelMendez is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@johnpyp