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

Second Logo and Contributors #33

Merged
merged 4 commits into from
Oct 2, 2020

Conversation

tylerkrupicka
Copy link
Contributor

What Changed

We had two awesome contributors add logos to the project, which is tough because we want to recognize both. The current plan is to leave the image logo in the ReadMe, but I've added the animated one so we can use it in the future maybe in the contributors file.

I also added both as contributors to the project to recognize their awesome work.

#27
#28

Why

Todo:

  • Add tests
  • Add docs
  • Add yourself to contributors (run yarn contributors:add)

@tylerkrupicka tylerkrupicka added documentation Changes only affect the documentation skip-release Preserve the current version when merged labels Oct 2, 2020
@tylerkrupicka tylerkrupicka merged commit d8afa63 into intuit:master Oct 2, 2020
@tylerkrupicka tylerkrupicka deleted the second-logo branch October 2, 2020 20:38
This was referenced Oct 2, 2020
@hipstersmoothie
Copy link
Contributor

🚀 PR was released in v2.1.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes only affect the documentation released skip-release Preserve the current version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants