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

Add some padding around the logo #411

Merged
merged 1 commit into from
May 24, 2021

Conversation

parthsarthy
Copy link
Contributor

Which issue does this PR close?

Related to #67

Rationale for this change

Adds spacing around the logo for dark mode
Before
Screenshot 2021-05-24 at 14 25 31

After
Screenshot 2021-05-24 at 14 25 20

What changes are included in this PR?

Adds spacing around the logo

Are there any user-facing changes?

No

@codecov-commenter
Copy link

Codecov Report

Merging #411 (d9f076f) into master (0aea0df) will increase coverage by 0.01%.
The diff coverage is 72.56%.

❗ Current head d9f076f differs from pull request most recent head 28a1723. Consider uploading reports for the commit 28a1723 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #411      +/-   ##
==========================================
+ Coverage   74.84%   74.86%   +0.01%     
==========================================
  Files         146      146              
  Lines       24382    24495     +113     
==========================================
+ Hits        18249    18338      +89     
- Misses       6133     6157      +24     
Impacted Files Coverage Δ
datafusion/src/scalar.rs 58.66% <72.56%> (+4.95%) ⬆️

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 0aea0df...28a1723. Read the comment docs.

@Dandandan Dandandan merged commit eeb69af into apache:master May 24, 2021
@Dandandan
Copy link
Contributor

Thank you @parthsarthy

@houqp houqp added the datafusion Changes in the datafusion crate label Jul 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
datafusion Changes in the datafusion crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants