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 new logos: PCMDI, PMP and CDAT #518

Merged
merged 3 commits into from
Nov 22, 2017
Merged

Conversation

durack1
Copy link
Collaborator

@durack1 durack1 commented Nov 21, 2017

Fix #517

@gleckler1 @lee1043 please merge when you can

@lee1043 lee1043 merged commit 2cad423 into master Nov 22, 2017
@durack1 durack1 deleted the issue517_durack1_UpdateLogos branch November 22, 2017 00:04
@doutriaux1
Copy link
Contributor

@durack1 @lee1043 i don'tthink you updated the setup.py so the new logos will not be distributed

@durack1
Copy link
Collaborator Author

durack1 commented Nov 22, 2017

@doutriaux1 good call, I hadn’t updated any of the scripts either so will reopen and address this issue. I’ll likely need some feedback for @lee1043 as to what PCMDI logo file fits best

This was referenced Nov 22, 2017
@durack1
Copy link
Collaborator Author

durack1 commented Nov 27, 2017

#518 (comment) should be solved in #519

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants