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

Try/inline block commenting #2

Conversation

akashdhawade2005
Copy link

What?

Why?

How?

Testing Instructions

Screenshots or screencast

youknowriad and others added 7 commits May 15, 2024 09:31
…dPress#61665)

Co-authored-by: youknowriad <youknowriad@git.wordpress.org>
Co-authored-by: jasmussen <joen@git.wordpress.org>
This commit improves the types in the dataviews package.

This is helpful by typing the `item` or `items` provided as arguments
to the functions of the fields and actions of the `DataViews`.

This commit does not implement requiring these types in the `DataView`
component, but helps to improve the types of the functions that are
used in the `DataView` package. It also allows exporting the types
to be used in consumers of the package.

- Add generic item type to all dataview types that operate
on `item` or lists of `items`.

- Update the usage in the dataviews package to incorporate
the use of the generic item type.

Co-authored-by: johnhooks <bitmachina@git.wordpress.org>
Co-authored-by: youknowriad <youknowriad@git.wordpress.org>
Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
* Calculate and report variance in performance results

* Calculate quartiles and outliers, report in table

* Fixup types

* Calculate stats in CI, too

* Format stats better

* Remove counts, unify formatting in reporter vs CI
Co-authored-by: youknowriad <youknowriad@git.wordpress.org>
Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
…oks doc. (WordPress#61597)

* Add a section about block_editor_settings_all.

Co-authored-by: ndiego <ndiego@git.wordpress.org>
Co-authored-by: juanmaguitar <juanmaguitar@git.wordpress.org>
* Update and restructure the Editor Hooks doc.

Co-authored-by: ndiego <ndiego@git.wordpress.org>
Co-authored-by: juanmaguitar <juanmaguitar@git.wordpress.org>
Copy link

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 props-bot label.

Unlinked Accounts

The following contributors have not linked their GitHub and WordPress.org accounts: @akashdhawade1991.

Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Unlinked contributors: akashdhawade1991.

Co-authored-by: youknowriad <youknowriad@git.wordpress.org>
Co-authored-by: johnhooks <bitmachina@git.wordpress.org>
Co-authored-by: jsnajdr <jsnajdr@git.wordpress.org>
Co-authored-by: ndiego <ndiego@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@poojabhimani12 poojabhimani12 merged commit 48ddd17 into poojabhimani12:try/inline-block-commenting May 15, 2024
35 of 62 checks passed
poojabhimani12 pushed a commit that referenced this pull request Sep 26, 2024
…ordPress#65289)

Co-authored-by: gigitux <gigitux@git.wordpress.org>
Co-authored-by: louwie17 <louwie17@git.wordpress.org>
Co-authored-by: youknowriad <youknowriad@git.wordpress.org>
Co-authored-by: oandregal <oandregal@git.wordpress.org>
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.

7 participants