Re: [PR] Add documentation to standard provider operators [airflow]

[via GitHub]

romsharon98 commented on PR #43716:
URL: https://github.com/apache/airflow/pull/43716#issuecomment-2466200324

   > > > > We we decided to combine datetime operators and sensors under 
operators documentation?
   > > > 
   > > > 
   > > > Sorry I’m not aware that, Do you think it’s worth combining everything 
into a single operator file? My main concern is that the file could become 
excessively large with all the content, and large file can be hard to read if 
any comes and see in my opinion.
   > > > But i am happy to merge, please suggest :)
   > > > I see one good thing sphinax is very smart :) its keeping all in one 
place when it generated even the content in different files under operator 
section, sample screenshot for the docs build.
   > > > <img alt="image" width="1091" 
src="https://private-user-images.githubusercontent.com/31437079/384268236-6319b189-ad4d-4551-b459-52185d6edada.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzExNTM4MjMsIm5iZiI6MTczMTE1MzUyMywicGF0aCI6Ii8zMTQzNzA3OS8zODQyNjgyMzYtNjMxOWIxODktYWQ0ZC00NTUxLWI0NTktNTIxODVkNmVkYWRhLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDExMDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQxMTA5VDExNTg0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ4MjdjMDkwYmEzODQ1MjdkMzRkMTFhZDFhMzg2OGUwMWFkMzkzNzk2OGM2MzAzMzlhYWMyM2IxMTIyMjRkODAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.Hdo0sNpjUzOweramHkYYFtsJVJIWpV_H75PJskdn-Z4";>
   > > 
   > > 
   > > What I ment is why we combined the sensors and the operators under the 
operators page? (and not keep a page for the sensors)
   > 
   > Oh okay sorry i miss understood the comment :). Yeah can do separate 
sections for sensors and operators? is it okay please confirm.
   
   Yes I think it's better to separate them like before and add the additional 
documentation to the correct sections 😄 


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org