-
Notifications
You must be signed in to change notification settings - Fork 110
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
Further update Optimization Detective documentation #1869
Conversation
|
7dee2de
to
8f10fe9
Compare
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## trunk #1869 +/- ##
=======================================
Coverage 66.70% 66.70%
=======================================
Files 88 88
Lines 7029 7029
=======================================
Hits 4689 4689
Misses 2340 2340
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current contents of the Google doc look good to me to move here. There may be small quirks here or there, but we can continue to finetune over time. This is excellent documentation and should provide a lot more context on how to use the plugin.
You probably need to still update the PR with the most recent changes from the doc, but I'm going to preemptively approve based on the doc contents.
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
See #1857