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

fix(docs): update timescale.png #24411

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Conversation

mathisve
Copy link
Contributor

SUMMARY

Timescale did a rebrand last month in which their logo changed. This PR reflects those changes to the logo. Here is a link to the official Timescale website announcing the rebrand.

BEFORE/AFTER LOGOS

Before:
timescale

After:
timescale

Thank you.

Signed-off-by: Mathis Van Eetvelde <[email protected]>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congrats on making your first PR and thank you for contributing to Superset! 🎉 ❤️

We hope to see you in our Slack community too! Not signed up? Use our Slack App to self-register.

@sfirke sfirke added the merge-if-green If approved and tests are green, please go ahead and merge it for me label Aug 5, 2024
@sfirke
Copy link
Member

sfirke commented Aug 5, 2024

Closing and reopening to try to trigger CI

@sfirke sfirke closed this Aug 5, 2024
@sfirke sfirke reopened this Aug 5, 2024
@github-actions github-actions bot added the doc Namespace | Anything related to documentation label Aug 5, 2024
@sfirke
Copy link
Member

sfirke commented Aug 5, 2024

@rusackas I'm trying to merge an old PR here and it looks good -- but I don't get why the linkinator plugin is failing. Do those two failing URLs make sense to you / do you know a way forward?

Copy link

codecov bot commented Aug 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.95%. Comparing base (fcf7cba) to head (c25ecf2).
Report is 2141 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #24411      +/-   ##
==========================================
+ Coverage   68.95%   69.95%   +0.99%     
==========================================
  Files        1903     1961      +58     
  Lines       74070    76779    +2709     
  Branches     8110     9039     +929     
==========================================
+ Hits        51077    53710    +2633     
+ Misses      20881    20850      -31     
- Partials     2112     2219     +107     
Flag Coverage Δ
javascript 58.36% <ø> (+2.73%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rusackas rusackas merged commit aa26ca5 into apache:master Aug 5, 2024
42 of 43 checks passed
@rusackas
Copy link
Member

rusackas commented Aug 5, 2024

I couldn't quite make sense of that either, but it's not a required check to merge.

WanjohiWanjohi pushed a commit to IDinsight/surveystream_superset_source that referenced this pull request Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Namespace | Anything related to documentation merge-if-green If approved and tests are green, please go ahead and merge it for me size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants