Skip to content

Add Dockerfile to .editorconfig and do a bit of housekeeping on the file#1049

Open
mhucka wants to merge 6 commits intotensorflow:masterfrom
mhucka:editorconfig-dockerfile
Open

Add Dockerfile to .editorconfig and do a bit of housekeeping on the file#1049
mhucka wants to merge 6 commits intotensorflow:masterfrom
mhucka:editorconfig-dockerfile

Conversation

@mhucka
Copy link
Copy Markdown
Member

@mhucka mhucka commented Apr 15, 2026

This PR makes some very minor updates to .editorconfig:

  • defines settings for Dockerfile
  • remove pattern for *.BUILD because this repo has no such files
  • streamline the comments some more

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .editorconfig file to refine style guide documentation and adjust indentation rules for Bazel and Docker files. The review feedback highlights the importance of retaining references to .clang-format to ensure consistency across C++ formatting tools and suggests consolidating redundant configuration blocks to improve maintainability.

Comment thread .editorconfig
Comment thread .editorconfig
@mhucka mhucka marked this pull request as ready for review April 15, 2026 01:52
@mhucka mhucka requested a review from MichaelBroughton April 15, 2026 02:23
@mhucka mhucka added the area/devops Involves build systems, Make files, Bazel files, continuous integration, and/or other DevOps topics label Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/devops Involves build systems, Make files, Bazel files, continuous integration, and/or other DevOps topics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants