#55646 closed task (blessed) (fixed)
Docblock improvements for 6.1
Reported by: | costdev | Owned by: | |
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch |
Focuses: | docs | Cc: |
Attachments (2)
Change History (140)
This ticket was mentioned in PR #2709 on WordPress/wordpress-develop by thejaydip.
3 years ago
#2
- Keywords has-patch added
Trac ticket: https://core.trac.wordpress.org/ticket/55646
SergeyBiryukov commented on PR #2709:
3 years ago
#4
Thanks for the PR! Merged in https://core.trac.wordpress.org/changeset/53411.
SergeyBiryukov commented on PR #2709:
3 years ago
#5
Thanks for the PR! Merged in https://core.trac.wordpress.org/changeset/53411.
This ticket was mentioned in PR #2766 on WordPress/wordpress-develop by whaze.
3 years ago
#7
Trac ticket: https://core.trac.wordpress.org/ticket/55646
#8
@
3 years ago
hello, it's my first contribution to WordPress, during the contribution's day at WCEU 2022
#9
@
3 years ago
@whaze thanks for the PR! i added a few changes to your proposal, now we're good to ship it into core :)
3 years ago
#11
Committed in https://core.trac.wordpress.org/changeset/53456
This ticket was mentioned in PR #2948 on WordPress/wordpress-develop by audrasjb.
2 years ago
#34
Trac ticket: https://core.trac.wordpress.org/ticket/55646
2 years ago
#39
Committed in https://core.trac.wordpress.org/changeset/53689
This ticket was mentioned in PR #3075 on WordPress/wordpress-develop by johnbillion.
2 years ago
#69
Trac ticket: https://core.trac.wordpress.org/ticket/55646
2 years ago
#70
Honestly I wasn't aware of numbered variable names before, but TIL: it's already used several times in Core, like parse_db_host()
for example.
Looks good to me at a glance but I think that if we're going to generalize this to many functions, we should probably open a Meta ticket to improve the docblock parser, as it looks pretty bugged on DevHub. For example, see https://developer.wordpress.org/reference/classes/wpdb/parse_db_host/
2 years ago
#71
Yeah there's at least a bug with $0
: https://developer.wordpress.org/reference/functions/wp_maybe_grant_site_health_caps/
Definitely something we should fix :)
johnbillion commented on PR #3075:
2 years ago
#72
Well spotted Jb, will you open a meta ticket?
2 years ago
#74
Meta ticket, for reference: https://meta.trac.wordpress.org/ticket/6433
This ticket was mentioned in PR #3082 on WordPress/wordpress-develop by johnbillion.
2 years ago
#77
Adds documentation for the arguments for registering a block style.
Trac ticket: https://core.trac.wordpress.org/ticket/55646
johnbillion commented on PR #3075:
2 years ago
#83
johnbillion commented on PR #3082:
2 years ago
#86
This ticket was mentioned in Slack in #core by desrosj. View the logs.
2 years ago
This ticket was mentioned in PR #3121 on WordPress/wordpress-develop by kebbet.
2 years ago
#90
Trac ticket: https://core.trac.wordpress.org/ticket/55646
This ticket was mentioned in PR #3122 on WordPress/wordpress-develop by kebbet.
2 years ago
#92
Trac ticket: https://core.trac.wordpress.org/ticket/55646
#93
@
2 years ago
Changes obejct
to object
in https://github.com/WordPress/wordpress-develop/pull/3122
2 years ago
#95
Thanks! Fixed in https://core.trac.wordpress.org/changeset/53939.
#97
@
2 years ago
What is the status for this PR, is anything blocking it?
https://github.com/WordPress/wordpress-develop/pull/3121
mukeshpanchal27 commented on PR #3121:
2 years ago
#98
@SergeyBiryukov Can you please check it as the changes make sense to merge?
robinwpdeveloper commented on PR #3121:
2 years ago
#99
LGTM 👍
2 years ago
#112
Committed in https://core.trac.wordpress.org/changeset/54199
This ticket was mentioned in PR #3364 on WordPress/wordpress-develop by kebbet.
2 years ago
#122
Use C
instead of С
in docblock for get_test_theme_version
.
#123
@
2 years ago
In #PR3365 the cyrillic letter С
is removed and the latin letter C
is added. See https://en.wikipedia.org/wiki/Es_(Cyrillic)
dream-encode commented on PR #3364:
2 years ago
#125
Thanks for the PR! Merged in https://core.trac.wordpress.org/changeset/54353.
In 53397: