This week (05/2021) on the Issue Tracker

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