WordPress.org
Get WordPress

Make WordPress Core

Opened 6 months ago

Closed 5 months ago

#63477 closed defect (bug) (fixed)

Unused import for utils within install.js

Reported by: yogeshbhutkar's profile yogeshbhutkar Owned by: jorbin's profile jorbin
Milestone: 6.9 Priority: normal
Severity: minor Version: 6.8
Component: Build/Test Tools Keywords: has-patch commit
Focuses: Cc:

Description

While exploring the codebase to understand how env is implemented, I came across an unused or duplicate import of utils.

Ref:
​https://github.com/WordPress/wordpress-develop/blob/97ed38471b8cc78019417d92efa28e3e1b478f0c/tools/local-env/scripts/install.js#L6

This import appears to be unnecessary and should ideally be cleaned up. It seems to have been introduced in [59279].

Change History (3)

This ticket was mentioned in ​PR #8835 on ​WordPress/wordpress-develop by ​@yogeshbhutkar.
6 months ago #1

  • Keywords has-patch added

Trac ticket: https://core.trac.wordpress.org/ticket/63477

#2 @mukesh27
6 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 6.9

Hi there,

Nice catch. It was introduce in [59279]

#3 @jorbin
5 months ago

  • Owner set to jorbin
  • Resolution set to fixed
  • Status changed from new to closed

In 60287:

Build/Test Tools: Remove unused import.

In [59279], this import was added but never used, likely since it was also imported as local_env_uitls on the next line.

Props yogeshbhutkar, mukesh27.
Fixes #63477. See #61218.

Note: See TracTickets for help on using tickets.