You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2020/07/01 05:52:29 UTC

[GitHub] [incubator-superset] ceohockey60 opened a new pull request #10212: Create INTHEWILD.md

ceohockey60 opened a new pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212


   #9438 # SUMMARY
   <!--- Describe the change below, including rationale and design decisions -->
   
   Create INTHEWILD.md to: 
   
   1. collect all Superset users into a single document and shorten README to make it more usable; 
   2. Categorize users into broad industry categories, which makes the document more useful as a reference for new users looking for industry peers and best practices (e.g. healthcare industry has unique needs and use cases); 
   3. removed some companies, whose website is inaccessible or defunct.
   
   After this file is created, will link to it from README. 
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   <!--- Skip this if not applicable -->
   
   ### TEST PLAN
   <!--- What steps should be taken to verify the changes -->
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ ] Changes UI
   - [ ] Requires DB Migration.
   - [ ] Confirm DB Migration upgrade and downgrade tested.
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] ceohockey60 commented on a change in pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
ceohockey60 commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r451898040



##########
File path: INTHEWILD.md
##########
@@ -0,0 +1,113 @@
+## Superset users in the wild
+
+Here's a list of organizations, broken down into broad industry categories, that have taken the time to send a PR to let
+the world know they are using Apache Superset. If you are a user and want to be recognized,
+all you have to do is file a simple PR [like this one](https://github.com/apache/incubator-superset/pull/7576). If you think

Review comment:
       Updated the PR example to a recent one




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] etr2460 commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
etr2460 commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-655575751


   Yeah, I think that's what @ceohockey60 was recommending


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] mistercrunch commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
mistercrunch commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-655297300


   I think we do not want the same info in 2 places. Maybe the README would have key (most recognizable) logos and a link to INTHEWILD
   
   ```markdown
   ## In the Wild
   dozens of [logos]
   
   see the full list and let the world know you use Superset [here](INTHEWILD.md)
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] mistercrunch commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
mistercrunch commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-653589493


   Totally fair @etr2460 . I was more concerned about people not discovering that they may want to add themselves to the list.
   
   I think that self-reported endorsement are strong statements that are nice to accumulate and surface, but I agree that `INTHEWILD.md` is the standard for that.
   
   This PR should probably also remove entries from `README.md` and link out to this new file.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] suddjian commented on a change in pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
suddjian commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r451771618



##########
File path: INTHEWILD.md
##########
@@ -0,0 +1,113 @@
+## Superset users in the wild
+
+Here's a list of organizations, broken down into broad industry categories, that have taken the time to send a PR to let
+the world know they are using Apache Superset. If you are a user and want to be recognized,
+all you have to do is file a simple PR [like this one](https://github.com/apache/incubator-superset/pull/7576). If you think

Review comment:
       Or replace it with another PR that adds a company to this file, once this has been merged in




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] mistercrunch commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
mistercrunch commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-652745523


   Personally I prefer having the list in the `README.md` so it's more discoverable (?)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] ceohockey60 commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
ceohockey60 commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-655182192


   Thank you for all the comments. So do we have agreement that we should have INTHEWILD as it is currently organized in a separate doc to link *out* from README? If so, I'll make the necessary changes in README, i.e. remove current list, add link from top nav section, etc. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] ceohockey60 commented on a change in pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
ceohockey60 commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r452314837



##########
File path: INTHEWILD.md
##########
@@ -0,0 +1,153 @@
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+
+## Superset users in the wild
+
+Here's a list of organizations, broken down into broad industry categories, that have taken the time to send a PR to let
+the world know they are using Apache Superset. If you are a user and want to be recognized,
+all you have to do is file a simple PR [like this one](https://github.com/apache/incubator-superset/pull/10122). If you think
+the categorization is inaccurate, please file a PR with your correction as well.
+Join our growing community!
+
+
+### Sharing Economy
+- [Airbnb](https://github.com/airbnb) 
+- [Faasos](http://faasos.com/) [@shashanksingh]
+- [Hostnfly](https://www.hostnfly.com/) [@alexisrosuel]
+- [Lime](https://www.limebike.com/) [@cxmcc]
+- [Lyft](https://www.lyft.com/)
+
+### Financial Services
+- [Aktia Bank plc](https://www.aktia.com) [@villebro]
+- [American Express](https://www.americanexpress.com) [@TheLastSultan]
+- [Capital Service S.A.](http://capitalservice.pl) [@pkonarzewski]
+- [Clark.de](http://clark.de/) 
+
+### Gaming
+- [Digit Game Studios](https://www.digitgaming.com/)
+- [Popoko VM Games Studio](https://popoko.live) 
+
+### E-Commerce
+- [AiHello](https://www.aihello.com) [@ganeshkrishnan1]
+- [Dragonpass](https://www.dragonpass.com.cn/) [@zhxjdwh]
+- [Fanatics](https://www.fanatics.com) [@coderfender]
+- [Fordeal](http://www.fordeal.com) [@Renkai]
+- [GFG - Global Fashion Group](https://global-fashion-group.com) [@ksaagariconic]
+- [HuiShouBao](http://www.huishoubao.com/) [@Yukinoshita-Yukino]
+- [Now](https://www.now.vn/) [@davidkohcw]
+- [Qunar](https://www.qunar.com/) [@flametest]
+- [Rakuten Viki](https://www.viki.com) 
+- [Shopee](https://shopee.sg) [@xiaohanyu]
+- [Shopkick](https://www.shopkick.com) [@LAlbertalli]
+- [Tails.com](https://tails.com) [@alanmcruickshank]
+- [THE ICONIC](http://theiconic.com.au/) [@ksaagariconic]
+- [Zalando](https://www.zalando.com) [@dmigo]
+- [Zalora](https://www.zalora.com) [@ksaagariconic]
+
+### Enterprise Technology
+- [Apollo GraphQL](https://www.apollographql.com/) [@evans]
+- [Astronomer](https://www.astronomer.io) [@ryw]
+- [Cloudsmith](https://cloudsmith.io) [@alancarson]
+- [CnOvit](http://www.cnovit.com/) [@xieshaohu]
+- [Deepomatic](https://deepomatic.com/) [@Zanoellia]
+- [Dial Once](https://www.dial-once.com/) 
+- [Dremio](https://dremio.com) [@narendrans]
+- [ELMO Cloud HR & Payroll](https://elmosoftware.com.au/)
+- [Endress+Hauser](http://www.endress.com/) [@rumbin]
+- [FBK - ICT center](http://ict.fbk.eu) 
+- [GfK Data Lab](http://datalab.gfk.com) [@mherr]
+- [GrowthSimple](https://growthsimple.ai/)
+- [Intercom](https://www.intercom.com/) [@kate-gallo]
+- [jampp](https://jampp.com/) 
+- [Konfío](http://konfio.mx) [@uis-rodriguez]
+- [Myra Labs](http://www.myralabs.com/) [@viksit]
+- [Nielsen](http://www.nielsen.com/) [@amitNielsen]
+- [Ona](https://ona.io) [@pld]
+- [Peak AI](https://www.peak.ai/) [@azhar22k]
+- [PeopleDoc](https://www.people-doc.com) [@rodo]
+- [Preset, Inc.](https://preset.io) 
+- [Pronto Tools](http://www.prontotools.io) [@zkan]
+- [PubNub](https://pubnub.com) [@jzucker2]
+- [Reward Gateway](https://www.rewardgateway.com) 
+- [ScopeAI](https://www.getscopeai.com) [@iloveluce]
+- [Showmax](https://tech.showmax.com) [@bobek]
+- [source{d}](https://www.sourced.tech) [@marnovo]
+- [Steamroot](https://streamroot.io/)
+- [Tenable](https://www.tenable.com) [@dflionis]
+- [timbr.ai](https://timbr.ai/) [@semantiDan]
+- [Tobii](http://www.tobii.com/) [@dwa]
+- [Tooploox](https://www.tooploox.com/) [@jakubczaplicki]
+- [Whale](http://whale.im)
+- [Windsor.ai](https://www.windsor.ai/) [@octaviancorlade]
+
+### Entertainment
+- [6play](https://www.6play.fr) [@CoryChaplin]
+- [bilibili](https://www.bilibili.com) [@Moinheart]
+- [Douban](https://www.douban.com/) [@luchuan]
+- [Kuaishou](https://www.kuaishou.com/) [@zhaoyu89730105]
+- [TME QQMUSIC/WESING](https://www.tencentmusic.com/) 
+- [Zaihang](http://www.zaih.com/)
+
+### Education
+- [Brilliant.org](https://brilliant.org/) 
+- [Udemy](https://www.udemy.com/) [@sungjuly]
+- [VIPKID](https://www.vipkid.com.cn/) [@illpanda]
+
+### Energy
+- [Airboxlab](https://foobot.io) [@antoine-galataud]
+- [DouroECI](http://douroeci.com/en/) [@nunohelibeires]
+- [Safaricom](https://www.safaricom.co.ke/) [@mmutiso]
+- [Scoot](https://scoot.co/) [@haaspt]
+
+### Healthcare
+- [Amino](https://amino.com) [@shkr]
+- [Living Goods](https://www.livinggoods.org) [@chelule]
+- [Maieutical Labs](https://maieuticallabs.it) [@xrmx]
+- [QPID Health](http://www.qpidhealth.com/) 
+- [TrustMedis](https://trustmedis.com) [@famasya]
+- [WeSure](https://www.wesure.cn/)
+
+### Others
+ - [Grassroot](https://www.grassrootinstitute.org/) 
+ - [komoot](https://www.komoot.com/) [@christophlingg]
+ - [Let's Roam](https://www.letsroam.com/) 
+ - [Twitter](https://twitter.com/) 
+ - [Yahoo!](https://yahoo.com/) 
+ 
+
+## License
+
+```
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+```

Review comment:
       cool removed.
   @suddjian please review when you have a sec. thank you!




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] codecov-commenter commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-652209665


   # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=h1) Report
   > Merging [#10212](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/c3cd63a2e84841b7a080c0531c528ad26fcd8190&el=desc) will **decrease** coverage by `11.02%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-superset/pull/10212/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff             @@
   ##           master   #10212       +/-   ##
   ===========================================
   - Coverage   70.45%   59.42%   -11.03%     
   ===========================================
     Files         594      404      -190     
     Lines       31498    13103    -18395     
     Branches     3223     3223               
   ===========================================
   - Hits        22193     7787    -14406     
   + Misses       9198     5136     -4062     
   - Partials      107      180       +73     
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | #cypress | `?` | |
   | #javascript | `59.42% <ø> (+<0.01%)` | :arrow_up: |
   | #python | `?` | |
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [superset-frontend/src/SqlLab/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9BcHAuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvQXBwLmpzeA==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/SqlLab/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9pbmRleC5qc3g=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/dashboard/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9BcHAuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvaW5kZXguanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/dashboard/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9pbmRleC5qc3g=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/setup/setupColors.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3NldHVwL3NldHVwQ29sb3JzLmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/chart/ChartContainer.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2NoYXJ0L0NoYXJ0Q29udGFpbmVyLmpzeA==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/setup/setupFormatters.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3NldHVwL3NldHVwRm9ybWF0dGVycy5qcw==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/reducers/index.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvcmVkdWNlcnMvaW5kZXguanM=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | ... and [328 more](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=footer). Last update [c3cd63a...dcb534d](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] ktmud commented on a change in pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
ktmud commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r452622453



##########
File path: README.md
##########
@@ -41,7 +41,7 @@ A modern, enterprise-ready business intelligence web application.
 [**Get Help**](#get-help) |
 [**Contributor Guide**](#contributor-guide) |
 [**Resources**](#resources) |
-[**Superset Users**](#superset-users) |
+[**Superset Users**](https://github.com/apache/incubator-superset/blob/master/INTHEWILD.md) |

Review comment:
       You can probably just use [relative path](https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/about-readmes#relative-links-and-image-paths-in-readme-files). GitHub will understand it just right.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] etr2460 merged pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
etr2460 merged pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] etr2460 commented on a change in pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
etr2460 commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r452608478



##########
File path: INTHEWILD.md
##########
@@ -0,0 +1,113 @@
+## Superset users in the wild
+
+Here's a list of organizations, broken down into broad industry categories, that have taken the time to send a PR to let
+the world know they are using Apache Superset. If you are a user and want to be recognized,
+all you have to do is file a simple PR [like this one](https://github.com/apache/incubator-superset/pull/7576). If you think

Review comment:
       We probably should just remove this link altogether, then add one back in once someone has edited this file post merge




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] etr2460 commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
etr2460 commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-652805894


   I think it's fair to say the README is too large right now (github.com/apache/incubator-superset takes up 11 screen heights for me) and we should try and optimize for only showing the most important content there.
   
   If the marketing that our user list provides is the most important part of having this in the README, I'd recommend something like a curated row of the logos of the most influential users in the README, with a link to INTHEWILD below. Superset's large userbase is evident by the repo's star count, and highlighting major users with logos in the README would both take up less space and increase the signal to noise ratio (for example I care more that Lyft uses Superset than a smaller startup).
   
   Thoughts @mistercrunch?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] codecov-commenter edited a comment on pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
codecov-commenter edited a comment on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-652209665


   # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=h1) Report
   > Merging [#10212](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/c3cd63a2e84841b7a080c0531c528ad26fcd8190&el=desc) will **decrease** coverage by `5.01%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-superset/pull/10212/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #10212      +/-   ##
   ==========================================
   - Coverage   70.45%   65.43%   -5.02%     
   ==========================================
     Files         594      598       +4     
     Lines       31498    32002     +504     
     Branches     3223     3239      +16     
   ==========================================
   - Hits        22193    20942    -1251     
   - Misses       9198    10879    +1681     
   - Partials      107      181      +74     
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | #cypress | `?` | |
   | #javascript | `59.31% <ø> (-0.11%)` | :arrow_down: |
   | #python | `69.70% <ø> (-0.45%)` | :arrow_down: |
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [superset-frontend/src/SqlLab/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9BcHAuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvQXBwLmpzeA==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/SqlLab/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9pbmRleC5qc3g=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/dashboard/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9BcHAuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvaW5kZXguanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/dashboard/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9pbmRleC5qc3g=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/setup/setupColors.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3NldHVwL3NldHVwQ29sb3JzLmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/chart/ChartContainer.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2NoYXJ0L0NoYXJ0Q29udGFpbmVyLmpzeA==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/setup/setupFormatters.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3NldHVwL3NldHVwRm9ybWF0dGVycy5qcw==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/reducers/index.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvcmVkdWNlcnMvaW5kZXguanM=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | ... and [199 more](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=footer). Last update [c3cd63a...a45e0ab](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] codecov-commenter edited a comment on pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
codecov-commenter edited a comment on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-652209665


   # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=h1) Report
   > Merging [#10212](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/c3cd63a2e84841b7a080c0531c528ad26fcd8190&el=desc) will **decrease** coverage by `5.03%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-superset/pull/10212/graphs/tree.svg?width=650&height=150&src=pr&token=KsB0fHcx6l)](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #10212      +/-   ##
   ==========================================
   - Coverage   70.45%   65.42%   -5.04%     
   ==========================================
     Files         594      598       +4     
     Lines       31498    32011     +513     
     Branches     3223     3239      +16     
   ==========================================
   - Hits        22193    20944    -1249     
   - Misses       9198    10886    +1688     
   - Partials      107      181      +74     
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | #cypress | `?` | |
   | #javascript | `59.31% <ø> (-0.11%)` | :arrow_down: |
   | #python | `69.68% <ø> (-0.48%)` | :arrow_down: |
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [superset-frontend/src/SqlLab/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9BcHAuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvQXBwLmpzeA==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/SqlLab/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL1NxbExhYi9pbmRleC5qc3g=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/dashboard/App.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9BcHAuanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvaW5kZXguanN4) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/dashboard/index.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2Rhc2hib2FyZC9pbmRleC5qc3g=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/setup/setupColors.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3NldHVwL3NldHVwQ29sb3JzLmpz) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/chart/ChartContainer.jsx](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2NoYXJ0L0NoYXJ0Q29udGFpbmVyLmpzeA==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/setup/setupFormatters.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL3NldHVwL3NldHVwRm9ybWF0dGVycy5qcw==) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | [superset-frontend/src/explore/reducers/index.js](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree#diff-c3VwZXJzZXQtZnJvbnRlbmQvc3JjL2V4cGxvcmUvcmVkdWNlcnMvaW5kZXguanM=) | `0.00% <0.00%> (-100.00%)` | :arrow_down: |
   | ... and [199 more](https://codecov.io/gh/apache/incubator-superset/pull/10212/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=footer). Last update [c3cd63a...a45e0ab](https://codecov.io/gh/apache/incubator-superset/pull/10212?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] ceohockey60 commented on a change in pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
ceohockey60 commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r453016057



##########
File path: README.md
##########
@@ -41,7 +41,7 @@ A modern, enterprise-ready business intelligence web application.
 [**Get Help**](#get-help) |
 [**Contributor Guide**](#contributor-guide) |
 [**Resources**](#resources) |
-[**Superset Users**](#superset-users) |
+[**Superset Users**](https://github.com/apache/incubator-superset/blob/master/INTHEWILD.md) |

Review comment:
       Gotcha. updated with relative path.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] etr2460 commented on a change in pull request #10212: docs: create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
etr2460 commented on a change in pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#discussion_r451921738



##########
File path: INTHEWILD.md
##########
@@ -0,0 +1,153 @@
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+
+## Superset users in the wild
+
+Here's a list of organizations, broken down into broad industry categories, that have taken the time to send a PR to let
+the world know they are using Apache Superset. If you are a user and want to be recognized,
+all you have to do is file a simple PR [like this one](https://github.com/apache/incubator-superset/pull/10122). If you think
+the categorization is inaccurate, please file a PR with your correction as well.
+Join our growing community!
+
+
+### Sharing Economy
+- [Airbnb](https://github.com/airbnb) 
+- [Faasos](http://faasos.com/) [@shashanksingh]
+- [Hostnfly](https://www.hostnfly.com/) [@alexisrosuel]
+- [Lime](https://www.limebike.com/) [@cxmcc]
+- [Lyft](https://www.lyft.com/)
+
+### Financial Services
+- [Aktia Bank plc](https://www.aktia.com) [@villebro]
+- [American Express](https://www.americanexpress.com) [@TheLastSultan]
+- [Capital Service S.A.](http://capitalservice.pl) [@pkonarzewski]
+- [Clark.de](http://clark.de/) 
+
+### Gaming
+- [Digit Game Studios](https://www.digitgaming.com/)
+- [Popoko VM Games Studio](https://popoko.live) 
+
+### E-Commerce
+- [AiHello](https://www.aihello.com) [@ganeshkrishnan1]
+- [Dragonpass](https://www.dragonpass.com.cn/) [@zhxjdwh]
+- [Fanatics](https://www.fanatics.com) [@coderfender]
+- [Fordeal](http://www.fordeal.com) [@Renkai]
+- [GFG - Global Fashion Group](https://global-fashion-group.com) [@ksaagariconic]
+- [HuiShouBao](http://www.huishoubao.com/) [@Yukinoshita-Yukino]
+- [Now](https://www.now.vn/) [@davidkohcw]
+- [Qunar](https://www.qunar.com/) [@flametest]
+- [Rakuten Viki](https://www.viki.com) 
+- [Shopee](https://shopee.sg) [@xiaohanyu]
+- [Shopkick](https://www.shopkick.com) [@LAlbertalli]
+- [Tails.com](https://tails.com) [@alanmcruickshank]
+- [THE ICONIC](http://theiconic.com.au/) [@ksaagariconic]
+- [Zalando](https://www.zalando.com) [@dmigo]
+- [Zalora](https://www.zalora.com) [@ksaagariconic]
+
+### Enterprise Technology
+- [Apollo GraphQL](https://www.apollographql.com/) [@evans]
+- [Astronomer](https://www.astronomer.io) [@ryw]
+- [Cloudsmith](https://cloudsmith.io) [@alancarson]
+- [CnOvit](http://www.cnovit.com/) [@xieshaohu]
+- [Deepomatic](https://deepomatic.com/) [@Zanoellia]
+- [Dial Once](https://www.dial-once.com/) 
+- [Dremio](https://dremio.com) [@narendrans]
+- [ELMO Cloud HR & Payroll](https://elmosoftware.com.au/)
+- [Endress+Hauser](http://www.endress.com/) [@rumbin]
+- [FBK - ICT center](http://ict.fbk.eu) 
+- [GfK Data Lab](http://datalab.gfk.com) [@mherr]
+- [GrowthSimple](https://growthsimple.ai/)
+- [Intercom](https://www.intercom.com/) [@kate-gallo]
+- [jampp](https://jampp.com/) 
+- [Konfío](http://konfio.mx) [@uis-rodriguez]
+- [Myra Labs](http://www.myralabs.com/) [@viksit]
+- [Nielsen](http://www.nielsen.com/) [@amitNielsen]
+- [Ona](https://ona.io) [@pld]
+- [Peak AI](https://www.peak.ai/) [@azhar22k]
+- [PeopleDoc](https://www.people-doc.com) [@rodo]
+- [Preset, Inc.](https://preset.io) 
+- [Pronto Tools](http://www.prontotools.io) [@zkan]
+- [PubNub](https://pubnub.com) [@jzucker2]
+- [Reward Gateway](https://www.rewardgateway.com) 
+- [ScopeAI](https://www.getscopeai.com) [@iloveluce]
+- [Showmax](https://tech.showmax.com) [@bobek]
+- [source{d}](https://www.sourced.tech) [@marnovo]
+- [Steamroot](https://streamroot.io/)
+- [Tenable](https://www.tenable.com) [@dflionis]
+- [timbr.ai](https://timbr.ai/) [@semantiDan]
+- [Tobii](http://www.tobii.com/) [@dwa]
+- [Tooploox](https://www.tooploox.com/) [@jakubczaplicki]
+- [Whale](http://whale.im)
+- [Windsor.ai](https://www.windsor.ai/) [@octaviancorlade]
+
+### Entertainment
+- [6play](https://www.6play.fr) [@CoryChaplin]
+- [bilibili](https://www.bilibili.com) [@Moinheart]
+- [Douban](https://www.douban.com/) [@luchuan]
+- [Kuaishou](https://www.kuaishou.com/) [@zhaoyu89730105]
+- [TME QQMUSIC/WESING](https://www.tencentmusic.com/) 
+- [Zaihang](http://www.zaih.com/)
+
+### Education
+- [Brilliant.org](https://brilliant.org/) 
+- [Udemy](https://www.udemy.com/) [@sungjuly]
+- [VIPKID](https://www.vipkid.com.cn/) [@illpanda]
+
+### Energy
+- [Airboxlab](https://foobot.io) [@antoine-galataud]
+- [DouroECI](http://douroeci.com/en/) [@nunohelibeires]
+- [Safaricom](https://www.safaricom.co.ke/) [@mmutiso]
+- [Scoot](https://scoot.co/) [@haaspt]
+
+### Healthcare
+- [Amino](https://amino.com) [@shkr]
+- [Living Goods](https://www.livinggoods.org) [@chelule]
+- [Maieutical Labs](https://maieuticallabs.it) [@xrmx]
+- [QPID Health](http://www.qpidhealth.com/) 
+- [TrustMedis](https://trustmedis.com) [@famasya]
+- [WeSure](https://www.wesure.cn/)
+
+### Others
+ - [Grassroot](https://www.grassrootinstitute.org/) 
+ - [komoot](https://www.komoot.com/) [@christophlingg]
+ - [Let's Roam](https://www.letsroam.com/) 
+ - [Twitter](https://twitter.com/) 
+ - [Yahoo!](https://yahoo.com/) 
+ 
+
+## License
+
+```
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+```

Review comment:
       you probably don't need the license here if it's in the comment above.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org

[GitHub] [incubator-superset] ceohockey60 commented on pull request #10212: Create INTHEWILD.md

Posted by GitBox <gi...@apache.org>.
ceohockey60 commented on pull request #10212:
URL: https://github.com/apache/incubator-superset/pull/10212#issuecomment-655827549


   Made all the changes in both INTHEWILD and README. 
   I think it's best to leave the logo discussion to later because: 1. we may need permission from each organization whose logo we want to display; 2. which org counts as "noteworthy" is something to discuss separately.
   
   PTAL. Thanks @etr2460.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org