Add missing logos for integrations #13717
Conversation
|
@mik-laj think please help me with the review... again 😄 |
|
One integration still has a checkerboard pattern. Can you also add a logo for singularity? |
|
yes @mik-laj, I will do that right now 😄 sorry about that. |
|
@mik-laj please check again 😃 |
|
One more chessboard left, but this time it was slightly less visible. |
|
@mik-laj replaced the azure file share logo. |
|
Perfect! |
|
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest master or amend the last commit of the PR, and push it with --force-with-lease. |
|
Thanks for helping with the review! 😀 @mik-laj |
|
Cool. I might even try to incorporate that into the Provider Readme's :) |
|
Let me know if I can help @potiuk thanks for the help lads! Cheers |
Added missing logos .png images and updated provider.yaml file to point to the path for the logo image. Currently, there are many logos on the Apache Site that do not have logo images. Closes #13110