Skip to content

Conversation

@stefanadranca
Copy link
Collaborator

Motivation:

Blank lines between code blocks make the code clearer - we want this for the generated code.

Modifications:

  • changed the rendering functions for protocols, extensions and structs to add blank lines between their functions and properties
  • changed the tests accordingly

Result:

The generated code is clearer.

Motivation:

Blank lines between code blocks make the code clearer - we want this for the generated code.

Modifications:

- changed the rendering functions for protocols, extensions and structs to
add blank lines between their functions and properties
- changed the tests accordingly

Result:

The generated code is clearer.
@stefanadranca stefanadranca requested a review from glbrntt February 1, 2024 14:58
Copy link
Collaborator

@glbrntt glbrntt left a comment

Choose a reason for hiding this comment

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

LGTM!

@glbrntt glbrntt merged commit d95ec7a into grpc:main Feb 1, 2024
@glbrntt glbrntt added the version/v2 Relates to v2 label Feb 1, 2024
glbrntt pushed a commit to glbrntt/grpc-swift that referenced this pull request Feb 5, 2024
Motivation:

Blank lines between code blocks make the code clearer - we want this for the generated code.

Modifications:

- changed the rendering functions for protocols, extensions and structs to
add blank lines between their functions and properties
- changed the tests accordingly

Result:

The generated code is clearer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version/v2 Relates to v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants