Pharo-dev

By message

pharo-dev@lists.pharo.org

February 2021

12 discussions

This week (05/2021) on the Issue Tracker
by Marcus Denker Feb. 5, 2021

Feb. 5, 2021

We merged 34 Pull Requests and closed 64 issues this week. Often one focuses on the pull requests. But even closed issue tracker entries that do not result in one PR did require work... even if it might be just spending 30 minutes trying to recreate the problem only to find that it has indeed been fixed already a long time ago. A complete list of the closed issues is available at https://github.com/pharo-project/pharo/pulse Here are the PRs: Debugger ======== - 2681 debugger step into does not go into accessors or methods with primitives #8456 https://github.com/pharo-project/pharo/pull/8456 - 8379-Debugger-Contextat-infinite-loop-when-pressing-Through-in-a-test-case-condition #8426 https://github.com/pharo-project/pharo/pull/8426 - Removing suspicious update code in the debugger inspector, and fixing variable update problem with recursive call #8430 https://github.com/pharo-project/pharo/pull/8430 - 8370 metalinks cannot be reinstalled if the image was saved while they were uninstalled #8448 https://github.com/pharo-project/pharo/pull/8448 Refactoring =========== - RB - Expose variable refactoring commands in source context #8472 https://github.com/pharo-project/pharo/pull/8472 - RB - Create new refactoring, to create accessors with lazy initialization #8464 https://github.com/pharo-project/pharo/pull/8464 - RB - Fix ExtractMethodTransformation #8465 https://github.com/pharo-project/pharo/pull/8465 - RB - Introduce Extract setUp refactoring #8466 https://github.com/pharo-project/pharo/pull/8466 Features ======== - Reorganize world menu #8414 https://github.com/pharo-project/pharo/pull/8414 Fixes ===== - Fix smart characters and add missing tests #8504 https://github.com/pharo-project/pharo/pull/8504 - TaRenameSlot-TestAndFix #8484 https://github.com/pharo-project/pharo/pull/8484 - Changeset-fileout-DNU #8486 https://github.com/pharo-project/pharo/pull/8486 - 692-Deprecation-exception-when-use-Open-Directory-Chooser-button-dialog-of-Iceberg-Settings-UI #8492 https://github.com/pharo-project/pharo/pull/8492 - 8458-Filelist-does-not-understand-readStream #8480 https://github.com/pharo-project/pharo/pull/8480 - Introduce asOpenInterval in Interval #8477 https://github.com/pharo-project/pharo/pull/8477 - 8450-ChangeSorter-should-open-and-work-with-new-Spec2 #8453 https://github.com/pharo-project/pharo/pull/8453 - 7987-Class-definition-should-not-be-evaluated-if-it-contains-a-syntax-error #8452 https://github.com/pharo-project/pharo/pull/8452 - Fix copy-paste to take into account real AST node #8449 https://github.com/pharo-project/pharo/pull/8449 Cleanups - Deprecations =================== Work related to deprecations.. as you might know, we can tag methods not only as deprecated, but add code to them which on-the-fly, at runtime, fixes the caller. This makes it much easier to evolve APIs. - Adding deprecations that were recommended by DepMiner (accepted by 3 reviewers) #8495 https://github.com/pharo-project/pharo/pull/8495 - Adding deprecations that were recommended by DepMiner (accepted by all 4 reviewers) #8494 https://github.com/pharo-project/pharo/pull/8494 - Adding transformation rules for two deprecated methods of Context. One of them was recommended by DepMiner but had to be modified #8510 https://github.com/pharo-project/pharo/pull/8510 - 7638-withSqueakLineEnding-should-be-moved-to-deprecated-package #8470 https://github.com/pharo-project/pharo/pull/8470 - 8471-Deprecate-OSSDL2WindowHandle #847 https://github.com/pharo-project/pharo/pull/8475 - 8467-Fuel-is-using-an-old-API-from-MultiByteFileStream #8468 https://github.com/pharo-project/pharo/pull/8468 Cleanups ======== And more cleanupsâ¦ Among other cleanups, we are still slowly fixing all the cases where a variable is defined but not used. Nearly there! when it is done, we add a release test and thus will never be able to merge code like that ever again. - DiskDirectoryEntry-remove-unused-classVar #8488 https://github.com/pharo-project/pharo/pull/8488 - Cleanup-unused-vars-JobProgressBarMorph #8498 https://github.com/pharo-project/pharo/pull/8498 - remove some unused class variables 1 #8433 https://github.com/pharo-project/pharo/pull/8433 - Classify one unclassified method in Trait #8487 https://github.com/pharo-project/pharo/pull/8487 - Clean the log file created on the disk by the test #8457 https://github.com/pharo-project/pharo/pull/8457 - 7889-GZipReadStream--fileIntoNewChangeSet-fullFileName #8447 https://github.com/pharo-project/pharo/pull/8447 - 7879-Clunky-interactive-but-bad-flow-for-fileIntoNewChangeSet #8493 https://github.com/pharo-project/pharo/pull/8493 - Move-ArgumentNamesTest-to-Debugger-Model-Tests-Core #8438 https://github.com/pharo-project/pharo/pull/8438 - 8454-Removing-more-users-of-oldDefinition #8455 https://github.com/pharo-project/pharo/pull/8455 Documentation ============== - 4372-Improve-ifNotNil-and-friends-comment-methods-to-indicate-that-they-are-optimized #8483 https://github.com/pharo-project/pharo/pull/8483 - Comment Interval>>#rangeIncludes: and more complete test #8446 https://github.com/pharo-project/pharo/pull/8446

1 0

Progress Report -> Refactoring Project - ( January 24 - 28 )
by Evelyn Cusi Lopez Feb. 1, 2021

Feb. 1, 2021

Hi all Last week I did these tasks: - Review PRs of clean unused variables ( https://github.com/pharo-project/pharo/pull/8425 , https://github.com/pharo-project/pharo/pull/8422) - Fix issue to reuse existing method in RBExtractMethodTransformation ( https://github.com/pharo-project/pharo/pull/8465 ). - Add new refactoring to generate accessors with lazy initialization ( https://github.com/pharo-project/pharo/pull/8464) - Review some issues and I help to close some were already fixed ( https://github.com/pharo-project/pharo/issues/6002 , https://github.com/pharo-project/pharo/issues/7108) - Fix issue to rename manifest class when rename a package, but I have problems with dependencies ( https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:RenameMani… ). - Add the first version of extract setUp refactoring ( https://github.com/pharo-project/pharo/pull/8466) Tasks for this week: - Fix dependencies problems of rename manifest issue. - Improve extract setUp refactoring. - Fix previews of extract method with arguments ( https://github.com/pharo-project/pharo/issues/5855) - Fix rename refactoring, this should rename only local methods ( https://github.com/pharo-project/pharo/issues/6366) - Fix protect instance variable issue Cheers, Evelyn

1 0