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

2146-Hardcode-class-rewriter-in-RB-class #6061

Merged
merged 2 commits into from Apr 2, 2020

Conversation

LaurineDargaud
Copy link
Contributor

fix: #2146
From 62 to 23 references to RBParseTreeRewriter with good name: #parseTreeRewriterClass

From 62 to 23 references to RBParseTreeRewriter with good name: #parseTreeRewriterClass
@LaurineDargaud LaurineDargaud changed the title 2146-Hardcode-class-rewriter-and-searcher-in-RB-class 2146-Hardcode-class-rewriter-in-RB-class Mar 31, 2020
@Ducasse Ducasse closed this Mar 31, 2020
@Ducasse Ducasse reopened this Mar 31, 2020
@Ducasse
Copy link
Member

Ducasse commented Mar 31, 2020

This is strange since it did not build.

@Ducasse Ducasse closed this Mar 31, 2020
@Ducasse Ducasse reopened this Mar 31, 2020
@Ducasse
Copy link
Member

Ducasse commented Mar 31, 2020

rekicking the build. There is something strange.

Copy link
Member

@guillep guillep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some infinite loops over here and there :)

src/Refactoring-Core/RBAddParameterRefactoring.class.st Outdated Show resolved Hide resolved
src/Refactoring-Core/RBRenameMethodRefactoring.class.st Outdated Show resolved Hide resolved
Copy link
Member

@guillep guillep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok!

@guillep
Copy link
Member

guillep commented Apr 2, 2020

Tests are fixed! There is a fleaky test only. I'll open an issue for that.

@guillep guillep merged commit 2535f64 into pharo-project:Pharo9.0 Apr 2, 2020
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

Successfully merging this pull request may close these issues.

RB classes should not hardcode class rewriter and searcher....
3 participants