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

Sat, 09 Nov 2024 23:55:55 -0800 


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

   > > > > > 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 😄
   
   Yeah agree, hope this is fine now.
   
   <img width="1614" alt="image" 
src="https://github.com/user-attachments/assets/68e114a6-1211-4204-94da-6d2cf487ce27";>
   
   


-- 
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