gtristan commented on code in PR #2060:
URL: https://github.com/apache/buildstream/pull/2060#discussion_r2415824025


##########
doc/source/handling-files/overlaps.rst:
##########
@@ -144,6 +144,29 @@ command contain an ``rm`` statement, as such:
 This would cause later builds of ``runtime-only.bst`` to no longer
 conflict on the given file.
 
+Changing the location

Review Comment:
   I believe it is also, separately documented in `ScriptElement` right ?
   
   There are a couple (slowly growing number) of features which cannot be 
directly implemented on the `Element` class but are common amongst abstract 
subclasses, it *can perhaps* make sense perhaps to add a page to consolidate 
documentation for these common features on a single page...
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to