Page MenuHomePhabricator
Differential D3395

[services] Adjust tabs to be two spaces in build_base_image.sh
ClosedPublic
Actions

Authored by ashoat on Mar 10 2022, 8:24 AM.
Tags
None
Referenced Files
F3354258: D3395.id10264.diff
Sat, Nov 23, 12:35 PM
Unknown Object (File)
Wed, Nov 13, 12:40 PM
Unknown Object (File)
Tue, Nov 5, 5:37 PM
Unknown Object (File)
Thu, Oct 31, 10:36 PM
Unknown Object (File)
Oct 14 2024, 12:10 AM
Unknown Object (File)
Oct 13 2024, 11:54 AM
Unknown Object (File)
Oct 13 2024, 11:54 AM
Unknown Object (File)
Oct 13 2024, 11:54 AM
View All Files
Subscribers
• adrian
atul
• benschac
• karol
tomek

Details

Reviewers
• jim
Commits
rCOMM4949e158866b: [services] Adjust tabs to be two spaces in build_base_image.sh
Summary

Minor nit, I left the comment after my accept in D3234 so it was probably missed.

Test Plan

N/A

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/two-space-tabs
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Mar 10 2022, 8:24 AM
Herald added subscribers: • benschac, • karol, atul and 2 others. · View Herald TranscriptMar 10 2022, 8:24 AM
Harbormaster completed remote builds in B7280: Diff 10264.Mar 10 2022, 8:29 AM
ashoat requested review of this revision.Mar 10 2022, 8:29 AM
• jim accepted this revision.Mar 10 2022, 11:38 AM

Oops, just updated my .editorconfig

This revision is now accepted and ready to land.Mar 10 2022, 11:38 AM
ashoat added a comment.Mar 10 2022, 12:18 PM

Wonder if we should check in an .editorconfig into the repo

Closed by commit rCOMM4949e158866b: [services] Adjust tabs to be two spaces in build_base_image.sh (authored by ashoat). · Explain WhyMar 10 2022, 12:22 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM4949e158866b: [services] Adjust tabs to be two spaces in build_base_image.sh.
• jim added a comment.Mar 10 2022, 12:28 PM
In D3395#91819, @ashoat wrote:

Wonder if we should check in an .editorconfig into the repo

Yes, do recommend

ashoat added a comment.Mar 10 2022, 12:32 PM

Made a task https://linear.app/comm/issue/ENG-860/introduce-editorconfig

Revision Contents
Changeset List

PathSize
services/
scripts/
4 lines

Diff 10264

View Options

services/scripts/build_base_image.sh

Loading...