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

Unified formatting aross all files #20

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

Conversation

peterkos
Copy link

@peterkos peterkos commented Apr 14, 2016

I've attempted to unify the code formatting across all the various .ahk (and .lua) files. I haven't had a chance to actually test it, so it may have a bug or two.

(The Github diff freaks out over formatting changes and thinks the code's been entirely re-written, but for the most part i've just been indenting lines).

Disclaimer: I have very little experience with the AHK scripting language. I personally prefer a slightly alternate code style, but this is how most of the code was, so it should adhere to that.

.gitattributes Outdated
*.dot diff=astextplain
*.DOT diff=astextplain
*.pdf diff=astextplain
*.docx diff=astextplain
Copy link

Choose a reason for hiding this comment

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

.gitattributes and .gitignore files should always be ignored

@ghost
Copy link

ghost commented May 14, 2016

4 usually works better

@Matesaktesak
Copy link

I prefer tabs! Spaces are stupid because you cannot move whole chunks of code at one.

@peterkos
Copy link
Author

peterkos commented Jan 5, 2019

@Matesaktesak I agree, but sometimes you gotta pick your battles. For a code repo I'll probably never see again, I'll lean to what people want. If it's my own though? Tabs, tabs, tabs.

(also this PR is old and out of date lol)

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.

2 participants