#55794 closed defect (bug) (fixed)
Update the $_old_files array for 6.0
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.0 | Priority: | normal |
| Severity: | normal | Version: | 6.0 |
| Component: | Upgrade/Install | Keywords: | has-patch commit dev-reviewed |
| Focuses: | Cc: |
Description
The $_old_files array needs to be updated for 6.0.
Running the check command from the Handbook in the Dry Run for 6.0 produced this:
$ svn diff --summarize https://core.svn.wordpress.org/tags/5.9 https://core.svn.wordpress.org/trunk | grep '^D' D https://core.svn.wordpress.org/tags/5.9/wp-content/themes/twentytwentytwo/assets/fonts/LICENSE.md
The license information for Twenty Twenty-Two fonts appears to have been moved to the theme's readme.txt in [53286] / #55433.
Attachments (1)
Change History (9)
This ticket was mentioned in Slack in #core by sergey. View the logs.
3 years ago
#3
@
3 years ago
- Keywords commit added
- Owner set to hellofromTonya
- Status changed from new to reviewing
When running:
svn diff --summarize https://core.svn.wordpress.org/tags/5.9 https://core.svn.wordpress.org/trunk | grep '^D'
I got the same result:
D https://core.svn.wordpress.org/tags/5.9/wp-content/themes/twentytwentytwo/assets/fonts/LICENSE.md
Marking for commit.
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
3 years ago
This ticket was mentioned in Slack in #core-themes by jcastaneda. View the logs.
3 years ago
Note: See
TracTickets for help on using
tickets.
Assuming the command in the ticket description is accurate, and I have no reason to believe otherwise, the patch looks good to me. Not a committer, so will still want double-review on this.