[GitHub] [airflow] Bowrna commented on pull request #19470: Add how-to Guide for MSSQL operators

[GitBox <gi...@apache.org>]

Bowrna commented on pull request #19470:
URL: https://github.com/apache/airflow/pull/19470#issuecomment-964330746


   > > @potiuk I have added the docs and i tried to commit the doc into repository and the pre-commit build fails
   > > ```
   > > - hook id: provider-yamls
   > > - exit code: 1
   > > 
   > > Checking integration duplicates
   > > Checking completeness of list of {sensors, hooks, operators}
   > >  -- {sensors, hooks, operators} - Expected modules(Left): Current Modules(Right)
   > > Checking for duplicates in list of {sensors, hooks, operators}
   > > Checking completeness of list of transfers
   > >  -- Expected transfers modules(Left): Current transfers Modules(Right)
   > > Checking for duplicates in list of transfers
   > > Checking connection classes belong to package
   > >  -- Checking providers: present in code(left), mentioned in /Users/sathishkannan/code/airflow/.github/ISSUE_TEMPLATE/airflow_providers_bug_report.yml (right)
   > > Checking doc files
   > >  -- Checking document urls: expected(left), current(right)
   > >     -- Items in the left set but not the right:
   > >        '/docs/apache-airflow-providers-microsoft-mssql/operators.rst'
   > > ```
   > > 
   > > 
   > >     
   > >       
   > >     
   > > 
   > >       
   > >     
   > > 
   > >     
   > >   
   > > I could not understand this failure message. Can you help me to understand this?
   > 
   > Note that it fails on `hook id: provider-yamls` so this gives you context into where the problem is. Something that is related to the provider yaml. Possibly you also edited a howto guide (`.rst` file) but you did not add a corresponded entry in the provider `.yaml`
   
   @eladkal thank you. I could understand the context of the problem in provider yaml. I have edited the `.rst` file. But I am not sure what I have to add in the provider `.yaml` file. Do I have to increase the version or make any changes to `.yaml` file for adding `.rst` file? If there is any help/guide about this, it would be useful for me to get started. 


-- 
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-unsubscribe@airflow.apache.org

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