Skip to content

[AIRFLOW-XXXX] Add config to welcome first time contributors#7071

Merged
potiuk merged 1 commit intoapache:masterfrom
kaxil:add-greetings
Jan 6, 2020
Merged

[AIRFLOW-XXXX] Add config to welcome first time contributors#7071
potiuk merged 1 commit intoapache:masterfrom
kaxil:add-greetings

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Jan 6, 2020

I have added a feature to boring-cyborg to greet new contributors when they open new PR and when their first PR gets merged.


Link to JIRA issue: https://issues.apache.org/jira/browse/AIRFLOW-XXXX

  • Description above provides context of the change
  • Commit message starts with [AIRFLOW-NNNN], where AIRFLOW-NNNN = JIRA ID*
  • Unit tests coverage for changes (not needed for documentation changes)
  • Commits follow "How to write a good git commit message"
  • Relevant documentation is updated including usage instructions.
  • I will engage committers as explained in Contribution Workflow Example.

(*) For document-only changes, no JIRA issue is needed. Commit message starts [AIRFLOW-XXXX].


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.

@kaxil kaxil requested a review from potiuk January 6, 2020 04:23
@kaxil kaxil requested a review from dimberman January 6, 2020 04:25
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

awesome !

@potiuk potiuk merged commit 828e4b8 into apache:master Jan 6, 2020
@kaxil kaxil deleted the add-greetings branch February 28, 2020 14:32
galuszkak pushed a commit to FlyrInc/apache-airflow that referenced this pull request Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments