-
-
Notifications
You must be signed in to change notification settings - Fork 353
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
2466-Remove-unused-variables #2468
2466-Remove-unused-variables #2468
Conversation
Fixes pharo-project#2466 For now it does not fix everything but it's a first step
I cannot commit class instance var removal because of pharo-vcs/iceberg#1168 :( |
I think I did not removed variables used for tests but I am not 100% sure. I'll wait the checks to verify. |
Some broken tests are related. I'll fix them next week. |
failing tests now seem to be the ones that fail always (not related to the change) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
this was not in "ignore" status? :P |
I corrected the tests failures so it was integrable. I just forgot to remove the ignore. |
Remove unused instance variables.
Fixes #2466
For now it does not fix everything but it's a first step