Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Release 1.19.0 #1485

Merged
merged 4 commits into from
Feb 21, 2024
Merged

Release 1.19.0 #1485

merged 4 commits into from
Feb 21, 2024

Conversation

Lee-W
Copy link
Contributor

@Lee-W Lee-W commented Feb 21, 2024 •
edited
Loading

Closes: #1476

@Lee-W Lee-W mentioned this pull request Feb 21, 2024
Closed
@codecov Codecov
Copy link

codecov bot commented Feb 21, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4f13be7) 98.33% compared to head (ba27d16) 98.33%.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1485   +/-   ##
=======================================
  Coverage   98.33%   98.33%           
=======================================
  Files          94       94           
  Lines        4637     4637           
=======================================
  Hits         4560     4560           
  Misses         77       77

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

pankajkoti
pankajkoti reviewed Feb 21, 2024
View reviewed changes
CHANGELOG.rst Outdated Show resolved Hide resolved
CHANGELOG.rst Show resolved Hide resolved
pankajkoti
pankajkoti approved these changes Feb 21, 2024
View reviewed changes
Copy link
Collaborator

@pankajkoti pankajkoti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, have 1 suggestion to mention in the Changelog the bump of min python version to 3.8

@Lee-W
Copy link
Contributor Author

Lee-W commented Feb 21, 2024

@pankajkoti The regression tests work fine on both the celery executor and the k8s executor. Please feel free to merge it after you finish the testing on you part :)

@pankajkoti
Copy link
Collaborator

@pankajkoti The regression tests work fine on both the celery executor and the k8s executor. Please feel free to merge it after you finish the testing on you part :)

I think we already tested the regression. I will test the released providers once for sanity later. I think we can proceed with merging and releasing.

@Lee-W Lee-W merged commit ea23093 into main Feb 21, 2024
23 checks passed
@Lee-W Lee-W deleted the release-1-19-0 branch February 21, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pankajkoti pankajkoti pankajkoti approved these changes

@phanikumv phanikumv Awaiting requested review from phanikumv phanikumv is a code owner

@pankajastro pankajastro Awaiting requested review from pankajastro pankajastro is a code owner

@sunank200 sunank200 Awaiting requested review from sunank200 sunank200 is a code owner

@utkarsharma2 utkarsharma2 Awaiting requested review from utkarsharma2 utkarsharma2 is a code owner

@vatsrahul1001 vatsrahul1001 Awaiting requested review from vatsrahul1001 vatsrahul1001 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Release astronomer-providers 1.19.0
3 participants
@Lee-W @pankajkoti @pankajastro