You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/12/23 17:24:42 UTC

[GitHub] [airflow] zhongjiajie opened a new pull request #6883: [AIRFLOW-6330] Show cli help when param blank or typo

zhongjiajie opened a new pull request #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883
 
 
   When enter Airflow cli with blank parameter
   or typo parameter or wrong parameter will
   show Airflow cli help just like enter
   `Airflow [command] -h` command
   
   Make sure you have checked _all_ steps below.
   
   ### Jira
   
   - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW/) issues and references them in the PR title. For example, "\[AIRFLOW-XXX\] My Airflow PR"
     - https://issues.apache.org/jira/browse/AIRFLOW-6330
     - In case you are fixing a typo in the documentation you can prepend your commit with \[AIRFLOW-XXX\], code changes always need a Jira issue.
     - In case you are proposing a fundamental code change, you need to create an Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)).
     - In case you are adding a dependency, check if the license complies with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
   
   ### Description
   
   - [x] Here are some details about my PR, including screenshots of any UI changes:
   
   When enter Airflow cli with blank parameter
   or typo parameter or wrong parameter will
   show Airflow cli help just like enter
   `Airflow [command] -h` command

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] potiuk commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568881742
 
 
   Nice one @zhongjiajie !

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/9d3f8de3575eb9bbc24b05ad4c1359e810833036?src=pr&el=desc) will **decrease** coverage by `0.3%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.61%   84.31%   -0.31%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32578    32465     -113     
   - Misses       5923     6040     +117
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/providers/amazon/aws/hooks/s3.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYW1hem9uL2F3cy9ob29rcy9zMy5weQ==) | `95.72% <0%> (-0.03%)` | :arrow_down: |
   | [airflow/gcp/hooks/bigquery.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvaG9va3MvYmlncXVlcnkucHk=) | `70.28% <0%> (ø)` | :arrow_up: |
   | ... and [2 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [9d3f8de...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/d0fbb523d617316a35461e6ee53781a7a716092c?src=pr&el=desc) will **decrease** coverage by `0.28%`.
   > The diff coverage is `60%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.62%   84.33%   -0.29%     
   ==========================================
     Files         679      679              
     Lines       38497    38501       +4     
   ==========================================
   - Hits        32578    32470     -108     
   - Misses       5919     6031     +112
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.42% <60%> (-2.42%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [d0fbb52...8261c25](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/9d3f8de3575eb9bbc24b05ad4c1359e810833036?src=pr&el=desc) will **decrease** coverage by `0.3%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.61%   84.31%   -0.31%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32578    32465     -113     
   - Misses       5923     6040     +117
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/providers/amazon/aws/hooks/s3.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9wcm92aWRlcnMvYW1hem9uL2F3cy9ob29rcy9zMy5weQ==) | `95.72% <0%> (-0.03%)` | :arrow_down: |
   | [airflow/gcp/hooks/bigquery.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9nY3AvaG9va3MvYmlncXVlcnkucHk=) | `70.28% <0%> (ø)` | :arrow_up: |
   | ... and [2 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [9d3f8de...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] kaxil commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
kaxil commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568568424
 
 
   Travis is sad:
   
   ```
   airflow/bin/cli.py:1002:8: C0111: Missing class docstring (missing-docstring)
   ```

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] potiuk merged pull request #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883
 
 
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] zhongjiajie commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
zhongjiajie commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568858484
 
 
   Finally CI green, PTAL @kaxil  @mik-laj @potiuk 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/d0fbb523d617316a35461e6ee53781a7a716092c?src=pr&el=desc) will **decrease** coverage by `0.28%`.
   > The diff coverage is `60%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.62%   84.33%   -0.29%     
   ==========================================
     Files         679      679              
     Lines       38497    38501       +4     
   ==========================================
   - Hits        32578    32470     -108     
   - Misses       5919     6031     +112
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.42% <60%> (-2.42%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [d0fbb52...8261c25](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] zhongjiajie commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
zhongjiajie commented on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568534370
 
 
   This patch change cli behavior like below 
   
   * **Main command** or just `Airflow`
   ```sh
   # Only airflow with blank parameter
   $ airflow
   
   usage: airflow [-h]
   {config,connections,dags,db,flower,kerberos,pools,roles,rotate_fernet_key,scheduler,sync_perm,tasks,users,variables,version,webserver,worker}
                           sub-command help
       config              Show current application configuration
       connections         List/Add/Delete connections
       dags                List and manage DAGs
       db                  Database operations
       flower              Start a Celery Flower
       kerberos            Start a kerberos ticket renewer
       pools               CRUD operations on pools
       roles               Create/List roles
       rotate_fernet_key   Rotate all encrypted connection credentials and
                           variables; see
                           https://airflow.readthedocs.io/en/stable/howto/secure-
                           connections.html#rotating-encryption-keys
       scheduler           Start a scheduler instance
       sync_perm           Update permissions for existing roles and DAGs
       tasks               List and manage tasks
       users               CRUD operations on users
       variables           CRUD operations on variables
       version             Show the version
       webserver           Start a Airflow webserver instance
       worker              Start a Celery worker nodeoptional arguments:
     -h, --help            show this help message and exit
   
   airflow command error: the following arguments are required: subcommand, see help above.
   
   
   # airflow [subcommands] with blank parameter
   $ airflow connections
   
   usage: airflow connections [-h] {list,add,delete} ...positional arguments:
     {list,add,delete}
       list             List connections
       add              Add a connection
       delete           Delete a connectionoptional arguments:
     -h, --help         show this help message and exit
   
   airflow connections command error: the following arguments are required: subcommand, see help above.
   ```
   
   * **typo** or **wrong** parameter
   ```sh
   $ airflow connections docker
   
   usage: airflow connections [-h] {list,add,delete} ...positional arguments:
     {list,add,delete}
       list             List connections
       add              Add a connection
       delete           Delete a connectionoptional arguments:
     -h, --help         show this help message and exit
   
   airflow connections command error: argument subcommand: invalid choice: 'docker' (choose from 'list', 'add', 'delete'), see help above.
   ```

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/d0fbb523d617316a35461e6ee53781a7a716092c?src=pr&el=desc) will **decrease** coverage by `0.28%`.
   > The diff coverage is `60%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.62%   84.33%   -0.29%     
   ==========================================
     Files         679      679              
     Lines       38497    38501       +4     
   ==========================================
   - Hits        32578    32470     -108     
   - Misses       5919     6031     +112
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.42% <60%> (-2.42%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [d0fbb52...8261c25](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/54fe2c3ce4bd83b3d98b875f17c68dbb2ddfc20e?src=pr&el=desc) will **decrease** coverage by `0.46%`.
   > The diff coverage is `60%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.62%   84.15%   -0.47%     
   ==========================================
     Files         679      679              
     Lines       38497    38501       +4     
   ==========================================
   - Hits        32578    32402     -176     
   - Misses       5919     6099     +180
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.42% <60%> (-2.42%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/executors/sequential\_executor.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9leGVjdXRvcnMvc2VxdWVudGlhbF9leGVjdXRvci5weQ==) | `56% <0%> (-32%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/utils/dag\_processing.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy91dGlscy9kYWdfcHJvY2Vzc2luZy5weQ==) | `80.95% <0%> (-7.05%)` | :arrow_down: |
   | [airflow/utils/sqlalchemy.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy91dGlscy9zcWxhbGNoZW15LnB5) | `89.83% <0%> (-6.78%)` | :arrow_down: |
   | [airflow/utils/helpers.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy91dGlscy9oZWxwZXJzLnB5) | `76.43% <0%> (-6.33%)` | :arrow_down: |
   | ... and [1 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [54fe2c3...29db8e0](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #6883: [AIRFLOW-6330] Show cli help when param blank or typo
URL: https://github.com/apache/airflow/pull/6883#issuecomment-568630642
 
 
   # [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=h1) Report
   > Merging [#6883](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=desc) into [master](https://codecov.io/gh/apache/airflow/commit/7d4925b8423bbe2543f2ce3ecc869ed16ce1f677?src=pr&el=desc) will **decrease** coverage by `0.07%`.
   > The diff coverage is `66.66%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/airflow/pull/6883/graphs/tree.svg?width=650&token=WdLKlKHOAU&height=150&src=pr)](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master    #6883      +/-   ##
   ==========================================
   - Coverage   84.39%   84.31%   -0.08%     
   ==========================================
     Files         680      680              
     Lines       38501    38505       +4     
   ==========================================
   - Hits        32492    32465      -27     
   - Misses       6009     6040      +31
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [airflow/bin/cli.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9iaW4vY2xpLnB5) | `93.05% <66.66%> (-2.54%)` | :arrow_down: |
   | [airflow/kubernetes/volume\_mount.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZV9tb3VudC5weQ==) | `44.44% <0%> (-55.56%)` | :arrow_down: |
   | [airflow/kubernetes/volume.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3ZvbHVtZS5weQ==) | `52.94% <0%> (-47.06%)` | :arrow_down: |
   | [airflow/kubernetes/pod\_launcher.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3BvZF9sYXVuY2hlci5weQ==) | `45.25% <0%> (-46.72%)` | :arrow_down: |
   | [airflow/kubernetes/refresh\_config.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9rdWJlcm5ldGVzL3JlZnJlc2hfY29uZmlnLnB5) | `50.98% <0%> (-23.53%)` | :arrow_down: |
   | [...rflow/contrib/operators/kubernetes\_pod\_operator.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9jb250cmliL29wZXJhdG9ycy9rdWJlcm5ldGVzX3BvZF9vcGVyYXRvci5weQ==) | `78.75% <0%> (-20%)` | :arrow_down: |
   | [airflow/jobs/backfill\_job.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9qb2JzL2JhY2tmaWxsX2pvYi5weQ==) | `90.43% <0%> (-1.45%)` | :arrow_down: |
   | [airflow/hooks/dbapi\_hook.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9ob29rcy9kYmFwaV9ob29rLnB5) | `91.73% <0%> (+0.82%)` | :arrow_up: |
   | [airflow/models/connection.py](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree#diff-YWlyZmxvdy9tb2RlbHMvY29ubmVjdGlvbi5weQ==) | `68.78% <0%> (+0.97%)` | :arrow_up: |
   | ... and [4 more](https://codecov.io/gh/apache/airflow/pull/6883/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=footer). Last update [7d4925b...9feaeba](https://codecov.io/gh/apache/airflow/pull/6883?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services