Skip to content

Move 5115's logo to the logo folder because Brian is stupid.#4

Merged
ErikBoesen merged 1 commit intoFIRSTMap:masterfrom
cruzsbrian:master
Aug 20, 2016
Merged

Move 5115's logo to the logo folder because Brian is stupid.#4
ErikBoesen merged 1 commit intoFIRSTMap:masterfrom
cruzsbrian:master

Conversation

@cruzsbrian
Copy link
Copy Markdown
Contributor

No description provided.

@ErikBoesen ErikBoesen merged commit 71e9f00 into FIRSTMap:master Aug 20, 2016
@ErikBoesen
Copy link
Copy Markdown
Member

Yeah, that would be the issue.

@cruzsbrian
Copy link
Copy Markdown
Contributor Author

Maybe you/we ought to normalize the size of the logos.

@ErikBoesen
Copy link
Copy Markdown
Member

I realized I forgot to include that in the README. They should be a maximum of 38px wide or tall. Would you like to add that?

@cruzsbrian
Copy link
Copy Markdown
Contributor Author

Um, I can't right now, but I will if you don't. Want to add me as a contributor?

@ErikBoesen
Copy link
Copy Markdown
Member

I can do it now, that's fine.

It's not that I don't trust you, but I generally only make people contributors who have done a very significant portion of the work. So not yet. You're welcome to keep doing pull requests.

@cruzsbrian
Copy link
Copy Markdown
Contributor Author

OK. That's reasonable.

@ErikBoesen
Copy link
Copy Markdown
Member

Again, no disrespect to you. I've been looking at your GitHub and the work you do looks incredible, so don't worry about it.

@ErikBoesen ErikBoesen added the logo Adding a team logo label Jul 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logo Adding a team logo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants