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.

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

adding change.org to the users list #2089

Closed
wants to merge 1 commit into from
Closed

adding change.org to the users list #2089

wants to merge 1 commit into from

Conversation

vijaykramesh
Copy link

Change.org has been using Airflow for a while now, would love to add our name to the list of companies using it!

Thanks.

@mention-bot
Copy link

@vijaykramesh, thanks for your PR! By analyzing the history of the files in this pull request, we identified @r39132, @mistercrunch and @jasonjho to be potential reviewers.

Copy link
Contributor

@artwr artwr left a comment

Choose a reason for hiding this comment

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

Please move the insert to the right alphabetical location :)

README.md Outdated
@@ -159,6 +159,7 @@ Currently **officially** using Airflow:
1. [Zendesk](https://github.com/zendesk)
1. [Zenly](https://zen.ly) [[@cerisier](https://github.com/cerisier) & [@jbdalido](https://github.com/jbdalido)]
1. [99](https://99taxis.com) [[@fbenevides](https://github.com/fbenevides), [@gustavoamigo](https://github.com/gustavoamigo) & [@mmmaia](https://github.com/mmmaia)]
1. [Change.org](https://www.change.org) [[@change](https://github.com/change), [@vijaykramesh](https://github.com/vijaykramesh)]
Copy link
Contributor

Choose a reason for hiding this comment

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

@vijaykramesh, we have tried to keep this alphabetized, any chance you could insert it with the "C"s ?

Otherwise, LGTM +1 welcome on board!

@vijaykramesh
Copy link
Author

There were a few others in the C section that were out of order, which I fixed as well.

@artwr
Copy link
Contributor

artwr commented Feb 18, 2017

@vijaykramesh Thanks! LGTM +1

@artwr
Copy link
Contributor

artwr commented Feb 18, 2017

Sorry, one last thing : I created a JIRA. Could you squash your commits and prepend your commit message with the JIRA name : AIRFLOW-885.

I will then merge your PR.

Thanks again.

@vijaykramesh
Copy link
Author

all set @artwr thanks!

@codecov-io
Copy link

codecov-io commented Feb 18, 2017

Codecov Report

Merging #2089 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2089   +/-   ##
=======================================
  Coverage   66.67%   66.67%           
=======================================
  Files         142      142           
  Lines       10725    10725           
=======================================
  Hits         7151     7151           
  Misses       3574     3574

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d93c6b...e4bf05b. Read the comment docs.

@asfgit asfgit closed this in a279be6 Feb 19, 2017
alekstorm pushed a commit to alekstorm/incubator-airflow that referenced this pull request Jun 1, 2017
Closes apache#2089 from
vijaykramesh/change/add_change_to_users_list
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