- Pharo-dev - lists.pharo.org

This week (18/2021) on the Pharo Issue Tracker
by Marcus Denker May 7, 2021

May 7, 2021

We merged 24 PRs last week and closed 24 issues. Refactoring Tools ================== - RB - Introduce new refactoring "Deprecated Class" and commands #9103 https://github.com/pharo-project/pharo/pull/9103 - RB - Fix move method changing protocols #9095 https://github.com/pharo-project/pharo/pull/9095 - Remove add argument refactor option from menu for message #9225 https://github.com/pharo-project/pharo/pull/9225 - RB - Fix namespace with aliased methods, and inline refactoring errors #9141 https://github.com/pharo-project/pharo/pull/9141 - RB - Add preview of refactorings, improve and add tests to Realize Class Refactoring #9146 https://github.com/pharo-project/pharo/pull/9146 - Improve "Set up score command" #9039 https://github.com/pharo-project/pharo/pull/9039 - RB - Clean code PR #9141 #9239 https://github.com/pharo-project/pharo/pull/9239 - RB - Improve rename method presenter #9234 https://github.com/pharo-project/pharo/pull/9234 - RB - Fix extract set up command #9235 https://github.com/pharo-project/pharo/pull/9235 - RB - Fix rename inst variable shortcut command #9233 https://github.com/pharo-project/pharo/pull/9233 - Fixes the issue #9250: The context menu now opens correctly #9251 https://github.com/pharo-project/pharo/pull/9251 Fixes ===== - Fix KMShortcutPrinter when single key is pressed #9214 https://github.com/pharo-project/pharo/pull/9214 - 7313 close all debuggers menu option is not working #9219 https://github.com/pharo-project/pharo/pull/9219 - Fix #9222: Cannot disable word wrap from Rubric #9224 https://github.com/pharo-project/pharo/pull/9224 - Fix-isSuperSend-Not-Hardcode-String #9236 https://github.com/pharo-project/pharo/pull/9236 - Fix-Compiler-bindings-self-super-thisContext #9242 https://github.com/pharo-project/pharo/pull/9242 Cleanups: Doc Comments ======================= The goal is to get doc comments (executable examples in method comments) to a state that we can run a CI regression test for them. - Fix-DocComments-Self-Reference #9237 https://github.com/pharo-project/pharo/pull/9237 - Broken-doc-comments-fix-errors #9218 https://github.com/pharo-project/pharo/pull/9218 - Fix some more doc comments #9246 https://github.com/pharo-project/pharo/pull/9246 Cleanups ======== - 9256-Failing-on-CI-ProperMethodCategorizationTesttestSetUpMethodInSUnitTestsNeedsToBeInRunningProtocol #9258 https://github.com/pharo-project/pharo/pull/9258 - fix-testNoUnusedInstanceVariablesLeft #9238 https://github.com/pharo-project/pharo/pull/9238 - testNoPeriodInMethodSignature-down-to-2 #9245 https://github.com/pharo-project/pharo/pull/9245 - cleaning-up-low-space-error #9249 https://github.com/pharo-project/pharo/pull/9249 - fixes: 9231 new* into newTopToBottom and newLeftToRight #9232 https://github.com/pharo-project/pharo/pull/9232

1 0

New file based HelpBrowser in Spec20
by Leo Frere May 4, 2021

May 4, 2021

Hello, For who doesn't know me, I'm Leo, I'm student intern from Lille and I worked with Stephane Ducasse. We worked on the NewHelpBrowser in Spec20 and we published a first version that you can use and find at the folowing link [ https://github.com/leofrere/NewTools-NewHelpBrowser | https://github.com/leofrere/NewTools-NewHelpBrowser ] . The new help browser is using microdown files on the disc and not strings stored into methods. In this version, you use a FileSystem with folder or Microdown file which can be internalized (it means that the physical files can be loaded in a virtual memory file system and the help can work without the physical file system). The differents files are parsed, converted to Pillar model and visited by a Topic Builder. The TopicBuilder creates NodeHelpTopic for every section. All the elements are covered with tests. In the git you have also a doc folder which used by the code in the comment class of the SpNewHelpBrowserPresenter. Here is the doc of Pharo 90 we started to migrate. We do know if the newHelpBrowser should replace the old one. Let us know. We will work on a BookBrowser to be able to read all Pharo books from Pharo. LÃ©o FRERE

1 0

Progress Report -> Refactoring Project - ( April 26 - April 30)
by Evelyn Cusi Lopez May 3, 2021

May 3, 2021

Hi everyone, Last week I did these tasks: -* Fix rename instance variable command*, Fix issue: Ctrl + R to rename a variable in a System Browser does not work #9215 <https://github.com/pharo-project/pharo/issues/9215> Check PR <https://github.com/pharo-project/pharo/pull/9233> to know the changes. - *Improve rename method presenter.* Fix issues: * Add Argument Refactoring generates a MNU in RBMessageNode <https://github.com/pharo-project/pharo/issues/9194> * When I rename a method I cannot rename its parameter <https://github.com/pharo-project/pharo/issues/9139> * Cannot extract method with arguments <https://github.com/pharo-project/pharo/issues/5855> Check PR <https://github.com/pharo-project/pharo/pull/9234> to know the changes. Video: https://youtu.be/mYeFbpMQhs8 - *Fix extract setUp command*. You can see the changes here: PR <https://github.com/pharo-project/pharo/pull/9235> . - *Clean code PR #9141*. Check changes here: PR <https://github.com/pharo-project/pharo/pull/9239> - *Add generate accessors with lazy initialization command*. Check changes here: PR <https://github.com/pharo-project/pharo/pull/9240> Video: https://youtu.be/SO0G6f1Y0fk - *Write post of*: New introduced refactorings in Pharo 9 <https://cusi-evelyn.medium.com/introduced-refactorings-in-pharo-9-9c7274b68…> Tasks for this week: - Finish to add the new preview to select a default scope. - Fix issue: RB - Inline all senders of class methods (instance creation) doesn't work, need improve pattern to search #9142 <https://github.com/pharo-project/pharo/issues/9142> - Fix issues: Enhance Refactoring Preview Widget #9060 <https://github.com/pharo-project/pharo/issues/9060> and Refactoring dialogs should not be modal #5894 <https://github.com/pharo-project/pharo/issues/5894> - Fix issue: RB - Improve instance variable refactoring #8665 <https://github.com/pharo-project/pharo/issues/8665> - Improve messages: No understable message #9229 <https://github.com/pharo-project/pharo/issues/9229> - Fix issue:Cmd+T shortcut doesn't work <https://github.com/pharo-project/pharo/issues/9241> Regards, Evelyn C.

1 0

[ANN] Freezing Pharo 9 development right now :)
by Esteban Lorenzano May 3, 2021

May 3, 2021

Hi, So, today I was checking how much it is missing for release and what is our status, and I realized if we do not freeze now, we will never release in a reasonable time. So, even if there are a couple of cleanups and that to still do that should happen before, I will declare that Pharo 9.0 enters in freeze Pharo 9.0 right now (and I will sneak the cleanups anyway, but being more careful). What does this means? No more features, just bugfixes No more fixes of things that can wait until Pharo 10 In two weeks I will open the Pharo 10 development branch, up to then, please help fixing what's missing (which is a lot) ! Have a great weekend, cheers! Esteban

3 2

class is not the same as classSide
by StÃ©phane Ducasse May 1, 2021

May 1, 2021

Dear contributors I would like to stress a key difference that some of you may have skipped. class is not equivalent to classSide. class jumps one level while clasSide arrives always to the same level the class. "Point class >>> Point classâ "Point class class >>> Metaclass" "Point classSide >>> Point class" "Point class classSide >>> Point class" So please consider that it is better to use the correct one. S -------------------------------------------- StÃ©phane Ducasse http://stephane.ducasse.free.fr / http://www.pharo.org 03 59 35 87 52 Assistant: Aurore Dalle FAX 03 59 57 78 50 TEL 03 59 35 86 16 S. Ducasse - Inria 40, avenue Halley, Parc Scientifique de la Haute Borne, BÃ¢t.A, Park Plaza Villeneuve d'Ascq 59650 France

1 0

This week (17/2021) on the Pharo Issue Tracker
by Marcus Denker April 30, 2021

April 30, 2021

We merged 37 Pull Requests and fixed 35 issues this week. The focus was for one on Newtools, the others have been general bugfixes and cleanups. NewTools 0.6.4 =============== - [spotter] enhance the way of keeping selection while adding new results - [inspector] halt on call appears again - [playground] add a simple versioning system based on ombu - fix #8803 (thanks @MarcusDenker) - fix pharo-spec/NewTools#154 - fix #888 - cleanups NewTools 0.6.5 ============== - spotter handles correctly scoping it by an origin (it will be used in calypso <meta+h> command, and probably others). Spec 0.8.11 ============= - feature: add standard dialogs - code presenter: enhance method selection - fix pharo-spec/Spec#1047 - fix #8434 (thanks @kasperosterbye) - fix #8421 (thanks @kasperosterbye) - some fixes in SpDemo (WIP) - cleanups Spec 0.8.12 ============ - trees now can collapse paths Fixes ===== - fixing-socket-availableData #9130 https://github.com/pharo-project/pharo/pull/9130 - 9196-Enter-key-in-the-numpad-does-not-work #9204 https://github.com/pharo-project/pharo/pull/9204 - 9208 rb sequence node periods statements temporaries point to the same instance #9209 https://github.com/pharo-project/pharo/pull/9209 - 9077-SpValueHolder-did-not-untderstand-asTest-in-new-playground-About-menu #9132 https://github.com/pharo-project/pharo/pull/9132 - Allow-Shadowing-RequestorVars #9202 https://github.com/pharo-project/pharo/pull/9202 - 9188-Text-does-understand-unescapeCharacter #9198 https://github.com/pharo-project/pharo/pull/9198 - fixes #9183 by adding a deprecated subclass #9191 https://github.com/pharo-project/pharo/pull/9191 - 9187-Debug-it-raises-isReturnSpecial-was-sent-to-nil #9190 https://github.com/pharo-project/pharo/pull/9190 - Context>>printDetails trimmed the last character #9134 https://github.com/pharo-project/pharo/pull/9134 - Fix 9150-Right-click-is-broken-on-variables-in-the-Calypso-browser #9151 https://github.com/pharo-project/pharo/pull/9151 - update commitish for newtools v0.6.3 #9133 https://github.com/pharo-project/pharo/pull/9133 - every AthensCanvas has own paint cache #9111 https://github.com/pharo-project/pharo/pull/9111 Cleanups ======== - testNoUnusedInstanceVariablesLeft-down-to-6 #9207 https://github.com/pharo-project/pharo/pull/9207 - RubParagraphDecorator-class-isUsed #9184 https://github.com/pharo-project/pharo/pull/9184 - Epicea-deprecation-rewrites #9192 https://github.com/pharo-project/pharo/pull/9192 - IconicListItem should be in Deprecated90 package #9180 https://github.com/pharo-project/pharo/pull/9180 - TabLabelItem should be in Deprecated90 package #9182 https://github.com/pharo-project/pharo/pull/9182 - AthensAbstractShape should be abstract #9158 https://github.com/pharo-project/pharo/pull/9158 - AthensAbstractMeshPaintPatch should be abstract #9155 https://github.com/pharo-project/pharo/pull/9155 - AthensAbstractPaint should be abstract #9153 https://github.com/pharo-project/pharo/pull/9153 - SignalLogger should be abstract #9163 https://github.com/pharo-project/pharo/pull/9163 - Implement CodeChunk(class)>>#isAbstract #9168 https://github.com/pharo-project/pharo/pull/9168 - Abstract classes in Keymapping package #9174 https://github.com/pharo-project/pharo/pull/9174 - BytecodeEncoder should implement #isAbstract to return true #9172 https://github.com/pharo-project/pharo/pull/9172 - Fix small lint issue in ClapImage commands #9166 https://github.com/pharo-project/pharo/pull/9166 - Remove class AthensTiger3 #9159 https://github.com/pharo-project/pharo/pull/9159 - 9160 review athens text display command #9161 https://github.com/pharo-project/pharo/pull/9161 - Cleanup BehaviorOwnedChunk #9170 https://github.com/pharo-project/pharo/pull/9170 - Fix typo in MicSingleLineBlock class comment #9178 https://github.com/pharo-project/pharo/pull/9178 - 8864/calypso names #9148 https://github.com/pharo-project/pharo/pull/9148 - Cleanup-CommentTestCase-cleanUpInstanceVariables #9143 https://github.com/pharo-project/pharo/pull/9143 - 9140-ContributingToTheCatalogHelp-can-be-removed #9144 https://github.com/pharo-project/pharo/pull/9144 - 9129-RubTextEditorunescapeCharacterinString-should-be-implemented-in-String #9131 https://github.com/pharo-project/pharo/pull/9131

1 0

[ANN] Next Pharo Sprint: April 30
by Marcus Denker April 27, 2021

April 27, 2021

Next Pharo Sprint: April 30 We will organize a Pharo sprint / Moose dojo April 30 Goals of this sprint: - Fix issues from tracker https://github.com/pharo-project/pharo/issues - Especially checking older issues This is a Remote Sprint, it happens on Discord. http://pharo.org/contribute-events We have added a Board to coordinate the work: https://github.com/orgs/pharo-project/projects/9 A list of all Pharo Event of the year an be found here: https://association.pharo.org/events

1 0

Progress Report -> Refactoring Project - ( April 19 - April 23)
by Evelyn Cusi Lopez April 26, 2021

April 26, 2021

Hi everyone, Last week I did these tasks: -* Introduce class deprecation refactoring*, Fix issue: Introduce class Deprecation as Refactoring #9033 <https://github.com/pharo-project/pharo/issues/9033> Check PR <https://github.com/pharo-project/pharo/pull/9103> to know the changes. You can see the new behavior in this video: https://youtu.be/sVq6cyyvf7s . -* Introduce **Migrate References From Class to Another*. Fix issue: MigrateReferencesFromClass: A to: NewClass #9101 <https://github.com/pharo-project/pharo/issues/9101>. Check PR <https://github.com/pharo-project/pharo/pull/9103> to know the changes. - *Add support to alias in RBNamespace *. Fix issue: RBNamespace doesn't support alias #9041 <https://github.com/pharo-project/pharo/issues/9041> . You can see the changes here: PR <https://github.com/pharo-project/pharo/pull/9141> . - *Improve inline all senders refactoring.* Fix issues: RB - Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 <https://github.com/pharo-project/pharo/issues/9042>, Inline senders refactoring does not correctly manage errors #6218 <https://github.com/pharo-project/pharo/issues/6218> . Check PR <https://github.com/pharo-project/pharo/pull/9141> to know the changes. You can watch the new behavior in: https://youtu.be/H9S3nmEHL9s . -* Improve preconditions of Realize Class Refactoring and add tests for it*. Check PR <https://github.com/pharo-project/pharo/pull/9146> to know the changes. You can watch the new behavior in this video: https://youtu.be/UAoiiEK1Xy4 . - *Add preview for refactorings.* Check PR <https://github.com/pharo-project/pharo/pull/9146> to know the changes. Tasks for this week: - Finish to add the new preview to select a default scope. - Fix issue: RB - Inline all senders of class methods (instance creation) doesn't work, need improve pattern to search #9142 <https://github.com/pharo-project/pharo/issues/9142> - Fix issues: Enhance Refactoring Preview Widget #9060 <https://github.com/pharo-project/pharo/issues/9060> and Refactoring dialogs should not be modal #5894 . <https://github.com/pharo-project/pharo/issues/5894> <https://github.com/pharo-project/pharo/issues/6218> - Fix issue: When I rename a method I cannot rename its parameter #9139 <https://github.com/pharo-project/pharo/issues/9139> - Fix issue: Inline temp refactoring can create code with different behavior. #6056 <https://github.com/pharo-project/pharo/issues/6056> (using create cascade refactoring) - Fix issue: RB - Improve instance variable refactoring #8665 <https://github.com/pharo-project/pharo/issues/8665> Regards, Evelyn C.

1 0

This week (16/2021) on the Pharo Issue Tracker
by Marcus Denker April 23, 2021

April 23, 2021

We merged 32 PRs and fixed 23 issue tracker entries this week. https://github.com/pharo-project/pharo/pulse Spec v0.8.10 ============ - added a trait SpTModel to add "model" behavior to presenters. - SpPanedLayout: deprecating position/position: using positionOfSlider: now - Fixed error when entering an empty text on the field of SpRGBSliders (Thanks @jordanmontt) - small fixes NewTools v0.6.3 ================ - Raw inspection should show always first fixed slots then variable slots (thanks @guillep) - do not use gtDisplay things (thanks @Ducasse) - applying deprecations - small fixes Refactoring Tools ================= - RB - Update deprecate method refactoring and add command #9085 https://github.com/pharo-project/pharo/pull/9085 - RB - Fix command extract set up #9109 https://github.com/pharo-project/pharo/pull/9109 Compiler ======== - Refactor semantics analyzer to not rely on #isLocalVariable message #9113 https://github.com/pharo-project/pharo/pull/9113 - Cleanup DoItVariable from locals related methods (markRead and etc.) #9122 https://github.com/pharo-project/pharo/pull/9122 - Compiler-Improve-lookupVar #9118 https://github.com/pharo-project/pharo/pull/9118 Features ======= - adding-variadic-TF-functions #9102 https://github.com/pharo-project/pharo/pull/9102 - Added a setting configuration for ClyTextLineNumberSwitchMorph #9104 https://github.com/pharo-project/pharo/pull/9104 - Pr new help #9090 https://github.com/pharo-project/pharo/pull/9090 - 4443-We-should-have-a-code-critic-rule-checking-for-dot-at-the-end-of-method-signature #9088 https://github.com/pharo-project/pharo/pull/9088 Fixes ===== - remove hidden character #9082 https://github.com/pharo-project/pharo/pull/9082 - Fix #8654 - Process>>#terminate should answer self consistently #8712 https://github.com/pharo-project/pharo/pull/8712 - 9069-Typing-in-Transcript-gives-a-DNU-doItContext-CompletionEngine #9084 https://github.com/pharo-project/pharo/pull/9084 - 5818-SHRBTextStyler-doesnt-format-comments-on-root-AST-node #9083 https://github.com/pharo-project/pharo/pull/9083 - Fix/9056: The 'meta' keyboard shortcut modifier should not be used in the UI #9078 https://github.com/pharo-project/pharo/pull/9078 - Order in Calypso menu order for test handling #9124 https://github.com/pharo-project/pharo/pull/9124 - Continue improving DrTests Doc comments plugin #9081 https://github.com/pharo-project/pharo/pull/9081 - Simplify-CommentTestCase #9116 https://github.com/pharo-project/pharo/pull/9116 Improve Tests ============= - Fixes: #9091: ading some tests to the Mock SUnit extensions #9092 https://github.com/pharo-project/pharo/pull/9092 - ReleaseTest-Dot-end-of-Signature #9097 https://github.com/pharo-project/pharo/pull/9097 - improve-taskit-tests-for-waitingGC #9106 https://github.com/pharo-project/pharo/pull/9106 - testNoUnusedInstanceVariablesLeft-down-to-12 #9126 https://github.com/pharo-project/pharo/pull/9126 - 6337-Ensure-test-classes-are-named-in-singular #9089 https://github.com/pharo-project/pharo/pull/9089 Cleanups ======== - 6319-SmallInteger-as31BitInteger-wrong-on-64-bit-systems #9100 https://github.com/pharo-project/pharo/pull/9100 - Editor-Cleanup-Unused-Rubric-Subclasses #9120 https://github.com/pharo-project/pharo/pull/9120 - Remove GTPlaygroundHelp #9079 https://github.com/pharo-project/pharo/pull/9079

1 0

Progress Report -> Refactoring Project - ( April 12 - April 16)
by Evelyn Cusi Lopez April 19, 2021

April 19, 2021

Hi everyone, Last week I did these tasks: - *Improve and add command to deprecate a method. *Check PR <https://github.com/pharo-project/pharo/pull/9085> to know the changes. - *Improve "Set up score command" *Check PR <https://github.com/pharo-project/pharo/pull/9039> to know the changes. I'm not sure if this issue is still relevant, but you can see the new behavior here: https://youtu.be/ejL_RTLPOMo . -* Fix move method with suggestion of protocol* (Move to package of defining class enh #8929 <https://github.com/pharo-project/pharo/issues/8929>) Check PR <https://github.com/pharo-project/pharo/pull/9095> to know the changes. You can see the new behavior in this video: https://youtu.be/r22E1BUZxrE . - *Add preview rename class refactoring*, now show changes one by one, not all in one. Check PR <https://github.com/pharo-project/pharo/pull/9043> to know the changes. -* Introduce class deprecation refactoring*, It was based mainly on Stef's suggestions, however before doing the PR I hope to resolve some doubts to see if any of the behaviors from the previous refactoring should be preserved. You can see the changes here: https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:ClassDepre… . - I start to do the *new preview to set the default scope*. Still in progress. - I continue fixing the issue: RBNamespace doesn't support alias #9041 <https://github.com/pharo-project/pharo/issues/9041> and Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 <https://github.com/pharo-project/pharo/issues/9042> . Still in progress. Tasks for this week: - Add command to class deprecation refactoring. - Finish to add the new preview to select a default scope. - Finish to fix: RBNamespace doesn't support alias #9041 <https://github.com/pharo-project/pharo/issues/9041> - Finish to fix issues: Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 <https://github.com/pharo-project/pharo/issues/9042> and Inline senders refactoring does not correctly manage errors #6218 <https://github.com/pharo-project/pharo/issues/6218> - Fix issues: Enhance Refactoring Preview Widget #9060 <https://github.com/pharo-project/pharo/issues/9060> and Refactoring dialogs should not be modal #5894 . <https://github.com/pharo-project/pharo/issues/5894> <https://github.com/pharo-project/pharo/issues/6218> Regards, Evelyn C.

2 2