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
Improve the method isValidGlobalName #7533
Improve the method isValidGlobalName #7533
Conversation
failing tests, e.g: FluidClassBuilderTest>>testCreateClassWithFullExpandedDefinitionKeepsTheMinimum |
self ifEmpty: [ ^ false ]. | ||
|
||
"reserverd default names" | ||
self = 'NameOfSubclass' ifTrue: [ ^ false ]. | ||
self = 'TNameOfTrait' ifTrue: [ ^ false ]. | ||
(#('NameOfSubclass' 'MyClass' 'TNameOfTrait') includes: self) ifTrue: [ ^ false ]. |
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.
I did not know about this logic. But it's very surprizing. Why we can't have classes with those names? What makes them so special?
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.
It would be really nice to explain it in the method comment
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.
@bergel also it should be "reserved" - not "reserverd"
@bergel there are broken tests and pending comments in the PR, and a couple of months of inactivity. I'll close it for now, please do not hesitate to reopen it. |
@guillep one can not reopen as outside contributor due to permissions. |
I can reopen it myself if required, no problem :) |
Fix #7532