Skip to content

Conversation

@edenlightning
Copy link
Contributor

@edenlightning edenlightning commented Nov 25, 2020

Fixes #4286

And remove unused assets.

@edenlightning
Copy link
Contributor Author

@teddykoker you are quick!!

Copy link
Collaborator

@Borda Borda left a comment

Choose a reason for hiding this comment

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

why are we removing also the other logos?

@edenlightning
Copy link
Contributor Author

Because they arent used anywhere

@Borda Borda modified the milestones: 1.0.x, 1.1 Nov 26, 2020
@codecov
Copy link

codecov bot commented Nov 27, 2020

Codecov Report

Merging #4844 (a0bd295) into master (c7e349e) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4844   +/-   ##
======================================
  Coverage      93%     93%           
======================================
  Files         124     124           
  Lines        9302    9302           
======================================
  Hits         8615    8615           
  Misses        687     687           

@SeanNaren SeanNaren added the ready PRs ready to be merged label Nov 29, 2020
Copy link
Contributor

@tchaton tchaton left a comment

Choose a reason for hiding this comment

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

LGTM !

@edenlightning edenlightning changed the title Replace lightning logo asset [CI SKIP] Replace lightning logo asset Dec 1, 2020
@tchaton tchaton merged commit 0b653b8 into Lightning-AI:master Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation related ready PRs ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update logo images

8 participants