Closed (fixed)
Project:
RobotsTxt
Version:
8.x-1.x-dev
Component:
Documentation
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 May 2020 at 13:02 UTC
Updated:
12 Oct 2023 at 19:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
tbsiqueiraWhen using Drupal Scaffold you need to add the following to your composer.json file:
Under the "extra" sections:
This will make composer ignore the robots.txt file
EDIT: Tip from Elisabeth Escribano
Comment #3
bradallenfisher commentedsweet tip!
Comment #4
ccjjmartin commentedThis is a great idea, I see two approaches:
1) We could take the easy route and update the README.md to include this as part of the installation section. Maybe add a "composer setup" subsection?
2) We could take the harder route and update the composer.json file to include this but then additionally we still have to update the documentation (README.md) to include changing the project specific composer.json to allow this module to overwrite its scaffolding settings.
So at a minimum this is an easy contribution for updating the README.md file. I am going to add the novice contributor tag.
The harder route here might not make sense because people still have to update their project repo. If anyone wants to see an example of allowing scaffolding in composer dependencies, I implemented it in this project: https://github.com/fourkitchens/sous-drupal-project/blob/2.x/composer.js... which pulls in the scaffolding settings from the distro: https://github.com/fourkitchens/sous-drupal-distro/blob/2.x/composer.jso...
Comment #6
pragati_kanade commentedComment #8
pragati_kanade commentedComment #9
ajv009 commentedLooks good to me
Comment #10
tyler36 commentedI created a patch that expands a little more.
I don't think we need include a reference to "locations" because this will be different between projects ( Eg. "composer install" vs Acquia DevDesktop2 ). Both of these projects already create the "locations" section by default.
Comment #11
ilgnerfagundes commentedI did a reading of the patch and everything is fine, it can be done the commit.
Comment #12
aiphesI just switch from composer 1 to 2 and adding the use of scaffolding. So when I delete the robots.txt file the site goes down, run
get back the website.
So do I need to apply the tips from #2 or just do an update of the module ? actually using the 1.4
thanks for help
Comment #13
ccjjmartin commentedThe latest patch does not apply. The wording on the latest patch looks good to me, just needs to be able to be applied.
Comment #15
rpayanmComment #16
rossb89 commentedComment #17
rossb89 commentedThe MR in #14 looks fine against 8.x-1.x but won't apply against 1.4 release (current latest release) as the README in that release is a .txt extension but in this commit here: https://git.drupalcode.org/project/robotstxt/-/commit/68297f33f472d5a210... it was changed to a markdown .MD file.
So marking as reviewed and tested as it's fine against the current 8.x-1.x branch.
Comment #18
orkutmuratyilmazAny chance for merge?
Comment #20
ccjjmartin commentedComment #21
ccjjmartin commentedThanks for the reminder to merge. Everyone on the thread got credit including the original mention of the user who suggested the tip originally. Thanks all.
Comment #23
ccjjmartin commented