`R` and recipes - Githubissues

RefactoringCombos / ArlosCommitNotation

A notation for small commits messages that show the risk involved in each step

310 stars 30 forks source link

`R` and recipes #14

Closed JayBazuzi closed 3 years ago

JayBazuzi commented 4 years ago

Why does R require:

You followed the published recipe, including running full-suite test runs when indicated.

Wouldn't a recipe give you r?

arlobelshee commented 4 years ago

Well, I'm meaning the recipes from Fowler. The ones for which proof rests on tests, not static analysis.

How can we make that clear?

JayBazuzi commented 4 years ago

I don't think "proof" is the right word here. Check? Verify? Not sure if changing that would help, though.

arlobelshee commented 3 years ago

I believe this is addressed in the current version, with more clarity about the distinction between "Provable refactorings" "Test-supported procedural refactorings" and "general remodeling". I'm going to close this as fixed. Let me know if you think I should re-open.