core/INSTALL.txt provides a command to copy files from the unpacked distribution into a base directory for your website. It lists drupal-x-y-z/.eslintrc as a file to move, but the actual filename is "eslintrc.json".
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | incorrect_filename_2888296.patch | 839 bytes | dinesh18 |
| #4 | Patch applied on 8.3.png | 6.34 KB | dinesh18 |
| #4 | Applied patch on 8.4.png | 6.24 KB | dinesh18 |
| #2 | incorrect_filename_2888296.patch | 839 bytes | jalpesh |
Comments
Comment #2
jalpesh commentedHere you go with patch.
Comment #3
cilefen commentedComment #4
dinesh18 commentedI have tested the patch and it seems to be cleanly applied on 8.3.x and 8.4.x
Attaching the git bash screengrab.
It seems the files which you were describing is .eslintrc.json and the patch corrects the file name.
Changing the status to RTBC.
For 8.4.x automatic testing, re-uploading the same patch
Comment #5
dinesh18 commentedRe-uploaded the same patch
Comment #6
dinesh18 commentedChanging the status to RTBC, the patch cleanly applied to 8.4.x.
I think we are good to go. :)
Comment #7
cilefen commentedI am updating credit.
Comment #10
cilefen commentedCommitted 5de007b on 8.4.x and cherry-picked to 8.3.x as 1a765c58a because this is a documentation improvement.
Thanks @jalpesh for the patch and @Dinesh18 for reviewing! A few words on procedure: We don't need confirmation that the patch applies, since the automated testing infrastructure tells us that. Screenshots of the patch applied to the codebase are not needed in any issue.
Is someone interested in looking into whether core/UPDATE.txt mentions all the correct files to copy when updating Drupal, then finding or opening an issue? And for that matter, does INSTALL.txt mention all relevant files?
Comment #11
dinesh18 commentedThanks @cilefen for the information.
Yes, I am interested. Let me know what I need to do to get started.