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

Indent #13

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Indent #13

wants to merge 3 commits into from

Conversation

IMbackK
Copy link

@IMbackK IMbackK commented Jan 11, 2021

run indent -linux -l 120 on entire codebase

Add the posibility to configure shortcuts inteded for the physical keys the
mapphones poses
@IMbackK
Copy link
Author

IMbackK commented Jan 11, 2021

commit #12 first

To use it, issue:

$ git blame --ignore-revs-file .git-blame-ignore-revs

To make it standard for a repo:

$ git config blame.ignoreRevsFile .git-blame-ignore-revs
@IMbackK
Copy link
Author

IMbackK commented May 7, 2021

@MerlijnWajer @parazyd what about this? thoughts?

@parazyd parazyd requested a review from freemangordon May 7, 2021 10:55
@parazyd
Copy link
Member

parazyd commented May 7, 2021

ACK

@MerlijnWajer
Copy link
Member

Did we settle on these params for most projects? I think it would be very helpful if we settle on some standards, and then perform these kind of commits. Don't mean to block this, but if we do, let's settle on the params. :)

@IMbackK
Copy link
Author

IMbackK commented May 7, 2021

no
so the maemo code base uses 2 different standarts.
Ones that use linux style (mostly low level stuff like mce and sutch)
and those that mostly use something kinda like GNU style (mostly high level stuff like systemui and hildon-desktop)
while the linux style programs are pretty coherent style wise the "GNU style" programs are really messy with some files doing tabs or spaces or tabs and spaces etc.

so since i really dont like gnu style and the gnu style code basses are way more messy i hereby suggest indent -linux -l 120

@MerlijnWajer
Copy link
Member

Maybe we should continue here: maemo-leste/bugtracker#329

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants