search

cmu-db / peloton

The Self-Driving Database Management System
http://pelotondb.io
Apache License 2.0
2.03k stars 624 forks source link

Remove Tabs! #1416

Closed apavlo closed 6 years ago

apavlo commented 6 years ago

This should have been caught in the code review!

https://github.com/cmu-db/peloton/blob/2676275c8ea298cb0fede43e2d9f838a0e6fb57e/src/include/catalog/catalog_cache.h#L53

db-ol commented 6 years ago

@apavlo My bad. I was wondering how you detected it on GitHub or you had to check in the raw file because tabs and spaces look the same for me as below.

wx20180619-092433 2x

@ksaito7 Could you please fix it? Thanks.

ksaito7 commented 6 years ago

Sorry!

tli2 commented 6 years ago

@ksaito7 https://github.com/cmu-db/peloton/wiki/Formatting this can help you. Also, I remember in CLion you can configure it to automatically format your code (Google C++ style is what we use)

ksaito7 commented 6 years ago

@til2 Thanks. I will use its script. And I set up my eclipse to use space.