astronomer / astro-cli

CLI that makes it easy to create, test and deploy Airflow DAGs to Astronomer
https://www.astronomer.io
Other
350 stars 70 forks source link

Stderr upgrade notice so it doesnt mess output #1636

Closed kushalmalani closed 5 months ago

kushalmalani commented 5 months ago

Description

Describe the purpose of this pull request.

Stderr upgrade notice so it doesnt mess output. If users are on an older cli version can avoid the warning notice interruption with command outputs by running the command and passing 2>/dev/null | head

๐ŸŽŸ Issue(s)

Related #1583, #1622, #1197

๐Ÿงช Functional Testing

List the functional testing steps to confirm this feature or fix.

๐Ÿ“ธ Screenshots

Add screenshots to illustrate the validity of these changes.

Screenshot 2024-04-28 at 15 01 06 Screenshot 2024-04-28 at 15 04 39 Screenshot 2024-04-28 at 15 04 48

๐Ÿ“‹ Checklist

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 86.73%. Comparing base (067e43c) to head (20e85f4).

Files Patch % Lines
version/version.go 0.00% 4 Missing :warning:
pkg/ansi/ansi.go 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1636 +/- ## ========================================== - Coverage 86.74% 86.73% -0.02% ========================================== Files 114 114 Lines 16671 16674 +3 ========================================== Hits 14462 14462 - Misses 1319 1322 +3 Partials 890 890 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.