search

Danny-Dunn / FRC-6978-rio

2 stars 0 forks source link

Add files via upload #16

Closed Juiceman9001 closed 1 year ago

Juiceman9001 commented 1 year ago

We should really start using this again like theres absolutely no reason not to especially if we leave code on people's personal laptops

Danny-Dunn commented 1 year ago

Are we not going to use classes? A 700 line code file is a bit ridiculous.

Danny-Dunn commented 1 year ago

Also we need to use classes so that multiple people can efficiently work on the code. It's way too easy to create conflicting changes with a single code file.

Juiceman9001 commented 1 year ago

I don't know how to use github yet lmao. We can tweak it later, I just thought something was better than nothing

On Sun, 26 Feb 2023 at 12:50, Danny-Dunn @.***> wrote:

Also we need to use classes so that multiple people can efficiently work on the code. It's way too easy to create conflicting changes with a single code file.

— Reply to this email directly, view it on GitHub https://github.com/Danny-Dunn/FRC-6978-rio/pull/16#issuecomment-1445419167, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASX4C5W5XY2PXM4JMB76KQLWZOJX5ANCNFSM6AAAAAAVISTY2M . You are receiving this because you modified the open/close state.Message ID: @.***>

Danny-Dunn commented 1 year ago

Yeah it's a good start we can restructure today.

Juiceman9001 commented 1 year ago

I won’t be there today but I’m sure it wont be too difficult. Also Matt’s not going to be there either if i remember correctly so depending on if ben and szander are there or not we can save it for tomorrow so you can get everyone up to speed on how it works.

Sent from my iPhone

On Feb 27, 2023, at 11:46, Danny-Dunn @.***> wrote:

 Yeah it's a good start we can restructure today.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you modified the open/close state.

Danny-Dunn commented 1 year ago

Yeah that makes more sense. I'll see if I'm in tmr.