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

Compatibility of code base with previous version not obvious #57

Open
jonathanfine opened this issue Oct 19, 2014 · 0 comments
Open

Compatibility of code base with previous version not obvious #57

jonathanfine opened this issue Oct 19, 2014 · 0 comments

Comments

@jonathanfine
Copy link
Contributor

@ghickman in https://github.com/py3minepi/py3minepi/pull/54#issuecomment-59644353 wrote

If you have qualms about areas of the code base which you believe are not backwards compatible then please raise them in a new issue. We are not looking to improve legacy code for the 1.0 release.

This is that issue, but not in exact form requested. Tony Hoare wrote

There are two ways of constructing a software design: One way is to make it so simple that there are obviously no deficiencies, and the other way is to make it so complicated that there are no obvious deficiencies. The first method is far more difficult.

The diff between the two codebases is full of noise, and has many changes that are not necessary for the purpose of backwards compatibility. As a result, I would say, there are no obvious deficiencies.

Or put another way, any deficiencies in the backwards compatibility are not obvious. There is evidence for this. Just 10 days ago @doismellburning added f7468b1, which was a reverted a change to broke the previously working intFloor.

Someone, it seems, decided to make a change to intFloor and it got committed. This change was only discovered after testing. It should have been obvious from a diff, and spotted. Are you, @ghickman, sure that all changes made are backwards compatible?

Tony Hoare wrote

The first method is far more difficult.

I believe in our case the first method is easier. It does however require discipline. I know we're itching to revise or rewrite the Javesque code.

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

No branches or pull requests

1 participant