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

Emit 'swift-format-ignore-file' in generated files to avoid having them reformatted by swift-format #1606

Merged

Conversation

smikhanov
Copy link
Contributor

Generator now emits a comment line in the generated code disabling processing of the generated file by the swift-format formatter. This is to allow checking in generated files into codebases that are regularly linted and reformatted using swift-format, without generating a change in generated files every time listing is done.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 31, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: smikhanov / name: Sergey Mikhanov (3f1cd3a)

@glbrntt glbrntt added the 🔨 semver/patch No public API change. label May 31, 2023
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.

Thanks @smikhanov.

Looks like this is failing the copyright header check, I think you'll need to update

drop_first=8
so that drop_first=9.

@smikhanov smikhanov force-pushed the disabling-swift-format-in-generated-code branch 2 times, most recently from cbbdaf6 to 7ee2c06 Compare May 31, 2023 13:17
@smikhanov smikhanov force-pushed the disabling-swift-format-in-generated-code branch from 7ee2c06 to dcf42e9 Compare May 31, 2023 13:21
@glbrntt glbrntt enabled auto-merge (squash) May 31, 2023 13:26
@glbrntt glbrntt merged commit fcef85f into grpc:main May 31, 2023
@smikhanov smikhanov deleted the disabling-swift-format-in-generated-code branch May 31, 2023 13:27
WendellXY pushed a commit to sundayfun/grpc-swift that referenced this pull request Aug 24, 2023
pinlin168 pushed a commit to sundayfun/grpc-swift that referenced this pull request Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants