abdulbasitds commented on a change in pull request #6007: [AIRFLOW-2310] Enable 
AWS Glue Job Integration
URL: https://github.com/apache/airflow/pull/6007#discussion_r385377395
 
 

 ##########
 File path: docs/integration.rst
 ##########
 @@ -17,7 +17,6 @@
 
 Integration
 ===========
-
 
 Review comment:
   Hi, I am sorry but resetting the commit and removing file doesnt make sense 
to me somehow (As reverting to some commit will still have the things that you 
wanted me to remove) and eventually make things complicated. I instead checked 
out this file from apache/master and comiit that like following, It doesnt 
appears in changed files as well, is it okay?
   
   ```
   git checkout apache/master -- docs/integration.rst
   git commit -m "removing a file from PR"
   git push origin master
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to