Skip to content

Conversation

@afragen
Copy link
Member

@afragen afragen commented Jul 28, 2022

A Feature Project for Plugin Dependencies has been created. Along with this a Feature Plugin has also been created.

Testing either via this PR or the feature plugin will show similar/identical results.

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


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

@afragen afragen force-pushed the dependencies-tab branch 2 times, most recently from d31e836 to 6b6ccb1 Compare July 28, 2022 00:55
@afragen
Copy link
Member Author

afragen commented Jul 28, 2022

Somehow I need to remove package-lock.json from this PR.

costdev and others added 26 commits February 5, 2024 04:14
…ze_during_tests

Plugin Dependencies: Require the class in Core's bootstrap, but just don't initialize it during tests.
…instead_of_creating_new_ones

Plugin Dependencies: Store reflected members instead of creating new ones in tests.
…_convert_to_slug

Plugin Dependencies: Add a test to ensure that `hello.php` is converted to `hello-dolly`.
Plugin Dependencies: Make modals persistent for Install Now, Activate and Update Now.
Plugin Dependencies: Fix issues reported by JSHint.
…in_data

Plugin Dependencies: Fix the key `get_plugins()` sets when updating plugin data.
Plugin Dependencies: Tests update
Plugin Dependencies: Fix circular dependencies
@costdev
Copy link
Contributor

costdev commented Feb 6, 2024

Thanks for the PR! Committed in r57545.

@costdev costdev closed this Feb 6, 2024
@afragen afragen deleted the dependencies-tab branch February 7, 2024 03:27
@afragen afragen restored the dependencies-tab branch February 7, 2024 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants