sync scheduler overhaul

[carykees98]

No description provided.

[lavajuno]

Good stuff, looking forward to getting this up and running.

[Warvan1]

Its hard to tell what parts of this is actually changing functionality because 99% of the diff is just changing the code formatting. (the new formatting is also harder to read than the old formatting)

[lavajuno]

I think a lot of this is refactoring to make it easier to add/change functionality later. We still need to do some work on manual sync before it's up and running, we're also experiencing a lot of #13 in production.

[Warvan1]

why did you change all the function definitions to the following format?

auto Schedule::getInstance() -> Schedule*
{
//code
}

this is WAY harder to read especially for newer students. Also I like to avoid auto (except for some cases like range for loops).

Also I am resisting the urge to nitpick all of the other weird formatting but I couldn't ignore this.

[aHalliday13]

I like the overall structure. I've highlighted a few formatting things that stood out to me, but once reaping is fixed I don't see any reason this couldn't be merged.

[aHalliday13]

Should we convert dry run to an environment variable to match sync port, or vice-versa?

[aHalliday13]

Should readme.md be capitalized

[aHalliday13]

As before, my comments are just on formatting/ consistency. After reading everyting over, and since it seems to be running fine on elephant, I don't see any reason this can't be merged.