Skip to content

Add extra options for slack webhook operator and slack hook#9409

Merged
dimberman merged 1 commit intoapache:v1-10-testfrom
benbenbang:enh/add_extra_option_slackwhoptr
Jun 23, 2020
Merged

Add extra options for slack webhook operator and slack hook#9409
dimberman merged 1 commit intoapache:v1-10-testfrom
benbenbang:enh/add_extra_option_slackwhoptr

Conversation

@benbenbang
Copy link
Copy Markdown
Contributor

@benbenbang benbenbang commented Jun 19, 2020


Make sure to mark the boxes below before creating PR: [x]


In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.

@benbenbang benbenbang changed the title ENH: Add extra options for slack webhook operator and slack hook Add extra options for slack webhook operator and slack hook Jun 19, 2020
Copy link
Copy Markdown
Member

@kaxil kaxil left a comment

Choose a reason for hiding this comment

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

Minor comment but LGTM otherwise

@kaxil kaxil changed the base branch from v1-10-stable to master June 20, 2020 00:46
@kaxil kaxil changed the base branch from master to v1-10-stable June 20, 2020 00:46
@kaxil kaxil added this to the Airflow 1.10.11 milestone Jun 20, 2020
@kaxil kaxil changed the base branch from v1-10-stable to v1-10-test June 20, 2020 00:47
@potiuk potiuk force-pushed the v1-10-test branch 8 times, most recently from f47e81a to e31761f Compare June 20, 2020 17:43
@benbenbang
Copy link
Copy Markdown
Contributor Author

Hi @kaxil, @mik-laj 👋
CI Build / Core:Pg9.6,Py3.7 failed without any logs, is it bcs time out? Thanks

@kaxil
Copy link
Copy Markdown
Member

kaxil commented Jun 22, 2020

I have restarted CI, let's see if it passes this time

@benbenbang
Copy link
Copy Markdown
Contributor Author

hmmm, it failed randomly... Do you have any thoughts on this @kaxil?

@dimberman dimberman merged commit 33973ab into apache:v1-10-test Jun 23, 2020
@boring-cyborg
Copy link
Copy Markdown

boring-cyborg bot commented Jun 23, 2020

Awesome work, congrats on your first merged pull request!

@benbenbang benbenbang deleted the enh/add_extra_option_slackwhoptr branch June 23, 2020 16:44
cfei18 pushed a commit to cfei18/incubator-airflow that referenced this pull request Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants