Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(mvp): Stage 1 - clean php includes #285

Merged
merged 2 commits into from
Apr 20, 2023

Conversation

johnhooks
Copy link
Collaborator

@johnhooks johnhooks commented Apr 19, 2023

What?

Remove unused code.

Why?

In preparation for 0.3.0 and integrating the work done in #259, It would be a good time to clean up code that we do not expect to use. Before drawing more attention to the repo, we should make sure it reflects the current direction of the project.

How?

  • Remove unused code and tests

@johnhooks johnhooks force-pushed the fix/clean-includes branch 2 times, most recently from 6f8ac24 to 5b50a52 Compare April 19, 2023 23:23
@johnhooks johnhooks requested review from erikyo and Sephsekla April 19, 2023 23:45
@johnhooks johnhooks added the [Scope] Model Definition of the data used in the WP Notify project label Apr 19, 2023
@johnhooks johnhooks changed the title fix: clean php includes of dead code and add new interfaces fix: clean php includes and add new model classes Apr 20, 2023
@johnhooks johnhooks force-pushed the fix/clean-includes branch from 0e81f62 to 33f6353 Compare April 20, 2023 13:16
@johnhooks johnhooks changed the title fix: clean php includes and add new model classes fix: clean php includes Apr 20, 2023
@johnhooks johnhooks changed the title fix: clean php includes fix: clean php includes - Stage 1 MVP Apr 20, 2023
@johnhooks johnhooks changed the title fix: clean php includes - Stage 1 MVP fix(mvp): Stage 1 - clean php includes Apr 20, 2023
@johnhooks johnhooks added php Pull requests that update Php code and removed [Scope] Model Definition of the data used in the WP Notify project labels Apr 20, 2023
@johnhooks johnhooks requested review from Sephsekla and erikyo and removed request for erikyo April 20, 2023 20:12
@johnhooks johnhooks merged commit 5020e2d into WordPress:develop Apr 20, 2023
@johnhooks johnhooks deleted the fix/clean-includes branch April 20, 2023 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants