Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add back rollback PRs #2225
Add back rollback PRs #2225
Changes from 5 commits
adf8355
b486803
b360741
07313a0
fee5de9
262f659
493a99a
d19c3fd
ba94fac
482886a
83c0233
67ab8f0
ab68649
e05a9a7
2b48ff3
028c2e4
93ac595
c057a94
3139a30
e991386
770fb1e
05a4dcc
fb70bb9
2c4e720
2beaa8e
65b39c7
c1bf50d
94960af
79c1cfd
cea0087
05c7983
0b76554
91266d7
b2bb225
26dc49e
6c0d28d
a310b42
b13a263
b6b9d77
8cf1717
6b75ff9
7551c6b
2546076
91e4538
546be25
8471773
d5b77b3
74383db
21ce71b
36fdb0b
0af20ed
acb3af0
5bc5f03
3c6f187
68d99b6
4da955d
c8d011f
07a6b62
4f67620
f78068a
ad94ef2
23a3f6a
fcc11de
13a7373
03f6055
5218153
9f4df74
a8985a7
999ef39
eb91389
88b7dd0
9ed6f4c
fa487ef
e64a906
4159d5e
31bbd88
55458f4
ca59139
47933c8
442c2d3
7eb1ab4
58a7e1d
c1405d8
0b892c2
cbbc839
f121f17
a0630a2
fe5d532
2d85876
6232186
4057a3b
3d97715
b29bc00
016918c
8e514cf
3378400
bafdfd9
9e73175
b175165
a90057a
af36bef
cc9fdd0
91c1d77
3d07f9f
df05c1f
fc21819
7ebf1a2
897d058
d5e6b19
d2c6fca
14189cc
cb15cbd
9654f47
1003200
62b6ce1
8d5c88c
04cbe46
7386bc9
71e42ba
56118e6
2cab593
8c1b405
c008aed
bf2202b
8fe3e31
3864a2d
5f7b01e
673971f
1d9f06a
130d462
3cf5a61
015aa86
692d10d
327314c
5854d50
d504977
3ad0a9f
9d314bd
302b608
34efea4
0834769
9c3cc6f
d362861
c1dd694
327c9e5
eebf7c6
4697d23
928121c
837320e
95efbbf
32f262e
e517fb8
f69ca91
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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
wp-content
portion of these may not be correct on sub-directory installs: eg a bedrock type config.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.
Shouldn't bedrock type systems set the
WP_CONTENT_DIR
which gets picked up in$wp_filesystem->wp_content_dir()
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.
Since all the tests use
$wp_content
, which should take into accountWP_CONTENT_DIR
, should we change all these references to use the variable?Does it really matter as the path that's listed in the descriptions doesn't take into account anything preceding in the path.
Is
wp/wp-content/...
substantially different in Site Health towp-content/...
?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.
As this is an extreme edge case and it's only in a path that everyone should understand, I say we leave and wait to see if someone complains.
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.
@peterwilsoncc can you test this? I've never used Bedrock or similar type of installation.
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.
I think this refers to the
wp-content
directory in general. There are some other similar instances in core:Looking at Bedrock, it appears to rename the
wp-content
directory toapp
. Not sure how to account for that in these messages, but maybe usingWP_CONTENT_DIR
would help?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.
Certainly not a blocker.