-
Notifications
You must be signed in to change notification settings - Fork 23
Removed colbyfayock references #180
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
Conversation
|
|
@aarya-16 is attempting to deploy a commit to the Cloudinary DevX Team on Vercel. A member of the Team first needs to authorize it. |
|
@devpatocld please take a look into this, and let me know. If you want me to add Cloudinary LTD in contributors in both the package.json changes, I can do that. There are also assets, images, and videos, which are probably from his Cloudinary cloud account, but I don't have the links to replace them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @aarya-16 ! Thank you for digging up all of these references to Colby. After some consideration, I don't think we actually want to remove him from the lists of contributors (.all-contributorssrc, the two README.mds, and the two package.jsons), as he has made significant contributions to the project, even though he is no longer active. But I think it is really important that we replace him from LICENSE (although I left a note about how exactly to refer to "Cloudinary") - thank you for catching that, and if you revert the other changes, I'll merge this straightaway.
Apologies for the hastily-filed issue which could have been better defined/scoped, and thank you for the contribution!
|
@all-contributors please add @aarya-16 for docs |
|
I've put up a pull request to add @aarya-16! 🎉 |
Description
Issue Ticket Number
Fixes #178
Type of change
Checklist