-
Notifications
You must be signed in to change notification settings - Fork 7
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
Processing UT changes to get possible property values #79
Comments
Like I said in google groups, We can't always expect there to be test changes with production code. If it is just a fixing of code and not a logic change or such, we still need to run regression on it and all it's properties to verify nothing changed or is broken. I feel we will get better results if we grab all possible values in the Test and not just the ones that changed. |
I am OK with this. Whatever only changed UTs or all UTs, we would implement the same function that grab the property info. And I think grabbing from all UTs is good. |
We could process the change of UT, find the changed method, and collect the property value from the config attribute.
Example PRs:
checkstyle/checkstyle#4905
checkstyle/checkstyle#4902
checkstyle/checkstyle#4853
checkstyle/checkstyle#4770
checkstyle/checkstyle#4736
The text was updated successfully, but these errors were encountered: