[wp-trac] [WordPress Trac] #54504: Update Requests library to version 2.0.0
WordPress Trac
noreply at wordpress.org
Mon Dec 6 20:53:07 UTC 2021
#54504: Update Requests library to version 2.0.0
-------------------------------------------------+-------------------------
Reporter: jrf | Owner:
| SergeyBiryukov
Type: task (blessed) | Status: reopened
Priority: normal | Milestone: 6.0
Component: External Libraries | Version:
Severity: normal | Resolution:
Keywords: php80 php81 has-patch has-unit- | Focuses:
tests early early-like-actually-early |
-------------------------------------------------+-------------------------
Comment (by azaozz):
Replying to [comment:19 schlessera]:
> As @jrf already stated, Requests is still considered an external, third-
party dependency
If it is under the "WordPress umbrella" it should follow the WordPress
best practices, i.e. backwards compatibility is a number one priority.
Frankly it is unthinkable that a library developed as part of WordPress
would not only break back-compat, but will cause fatal errors when used in
core...
> The correct way to name it according to the code style in place in the
repository
If there is such "code style" requirement in that repository it should be
removed as incompatible with WordPress. Also, generally, namespaces should
be considered "evil" in open source software and avoided as much as
possible. The reason it that they promote "lazy" names that go against the
best practice in writing good code (poor self-documenting, confusing
names, etc.).
Seems a possible/good way forward would be to release Requests 3.0 that
will be incompatible with 2.0 but fully compatible with 1.0, then consider
it for adding to the next WordPress release. The alternative would be to
move the Requests repository out of WordPress and continue developing it
as a "true" third-party library that is, unfortunately, incompatible with
WordPress.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/54504#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list