-
Notifications
You must be signed in to change notification settings - Fork 534
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
Suggestion: Pick a style guide for us to help clean up the code, without you having to stick to it yourself #140
Comments
This is a good idea and all, but I'm very afraid of letting other people "clean up the code" as you say. And that's the second time that something has gone horribly wrong with me using github. The first time, I ended up nearly losing like a month of changes to my scripts because I did SOMETHING wrong when "syncing" with github, and it downloaded a lot of old code from github, and directly overwrote most of my .ahk files. Fortunately, Notepad++ still had them in memory, so I saved those newer copies of everything somewhere else, and then did a compare on every single pair, checking to see what the hell happened, and which changes were from other people, which changes were from me, and manually, miserably putting it all back together. So what I do now, is if someone recommends a change on github, I'll just copy/paste it into my scripts. Code ONLY gets uploaded TO github. I have no idea how to safely download/sync/merge/pull/whatever-it's-called FROM github. I think you underestimate just how much you know, and just how little I know! I know it's sloppy, but it all currently WORKS, damn it, and many of my attempts to clean stuff up have resulted in miserable debugging sessions. I don't really have time to learn a lot of the stuff that I'd need to know to make this work. So.... don't submit pull requests. I won't really know what to do with them. Or, again, I'll just copy/paste the code directly. But I won't do that if it's too complicated, requiring me to do a lot of testing to make sure it'll be okay. I don't know how this works! I could learn, but in my experience, there's always some goddamned edge case that comes up and screws up everything, and then the real programmers are like, "oh yeah well of COURSE it works like THAT, this ruined repo is your fault, LOL." I do appreciate your comments, and will watch those videos you linked. Eventually. |
Alrighty
What dat mean?
I've made a repo on my own account, for you to observe and experiment with: https://github.com/Jonta/Taran-should-trash-this That way, you don't have to deal with the headache of explaining its existence to people coming here from e.g. YouTube I'll have a look at GitHub's own GUI https://desktop.github.com/ - Unsure whether I can install it on the Windows-machine I have access to right now. Edit: Reason for me taking a look: To better know what to recommend to you GitKraken has a nice visualisation of the tree, but only has a 7-day-free-trial Walking on eggshells sucks, especially when those eggshells are so important to your life. We'll find the right spellbooks to turn those puny wooden eggs into stone, then iron, then diamond, and finally netherite |
;comments like this
; comments like this
Reading through the current https://github.com/TaranVH/2nd-keyboard/blob/master/ALL_MULTIPLE_KEYBOARD_ASSIGNMENTS.ahk there are a whole bunch of different styles
I'd start submitting pull requests
But: Maybe you do have some preferences with regard to style, and that will lead to me wasting a bunch of time, and you feeling bad for me
Amazing features of picking a style guide:
There are a bunch of style guides out there, so if you're not picky, this one looks okay: https://github.com/aviaryan/Ahk-Best-Practices
The text was updated successfully, but these errors were encountered: