CUP Formatting - Githubissues

KoffeinFlummi / AGM_Compatibility

Compatibility PBOs for AGM

Other

7 stars 2 forks source link

CUP Formatting #10

Closed jonpas closed 9 years ago

jonpas commented 9 years ago

Merge if you want, leave otherwise, as discussed in #7

One question as well, all the empty classes, do they need to be there? I guess for easy of updating, seeing what has changed? But could something brake if CUP changes it or something?

KoffeinFlummi commented 9 years ago

I'd say yes.

nicolasbadano commented 9 years ago

One question as well, all the empty classes, do they need to be there? I guess for easy of updating, seeing what has changed? But could something brake if CUP changes it or something?

The purpose with that was:

Making it easier to gradually fill in the entries needed for various things
Documenting the current class inheritance.
Hopefully getting early "Updated base class" errors if CUP changes things, so we can detect changes and fix for them.

jonpas commented 9 years ago

Good plan!