[1.0.0-beta2] Rename *.ui_icons.yml to *.icons.yml
>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3480910. --> Reported by: [pdureau](https://www.drupal.org/user/1903334) Related to !39 >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>Because <code>Drupal\ui_icons\Plugin\IconPackManager</code> is in the scope of <span class="drupalorg-gitlab-issue-link project-issue-status-info project-issue-status-7"><a href="https://www.drupal.org/project/drupal/issues/3471494" title="Status: Closed (fixed)">#3471494: Add an icon management API</a></span></p> <p>And because we don't want to merge any "UI Icons" branding in Core. Let's rename the render element from <code>*.ui_icons.yml</code> to <code>*.icons.yml</code>.</p> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Rename, check where all the places where it is used (including tests):</p> <pre>&nbsp; protected function getDiscovery(): DiscoveryInterface {<br>&nbsp;&nbsp;&nbsp; if (!$this-&gt;discovery) {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; $this-&gt;discovery = new YamlDiscovery('ui_icons', $this-&gt;moduleHandler-&gt;getModuleDirectories() + $this-&gt;themeHandler-&gt;getThemeDirectories());<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; $this-&gt;discovery = new ContainerDerivativeDiscoveryDecorator($this-&gt;discovery);<br>&nbsp;&nbsp;&nbsp; }<br>&nbsp;&nbsp;&nbsp; return $this-&gt;discovery;<br>&nbsp; }</pre><h3 id="summary-remaining-tasks">Remaining tasks</h3> <p>Is there other "ui icons" stuff we need to rename before the MR?</p>
issue