remove unnecessary png file #16860

pull GChuf wants to merge 1 commits into bitcoin:master from GChuf:removepng changing 2 files +1 −1

GChuf commented at 8:55 PM on September 12, 2019: contributor

No description provided.
DrahtBot added the label Build system on Sep 12, 2019
DrahtBot added the label Docs on Sep 12, 2019
remove unnecessary png file bfd5fa716f
MarcoFalke commented at 2:53 AM on September 13, 2019: member

NACK, this is too large http://www.doxygen.nl/manual/config.html#cfg_project_logo
fanquake closed this on Sep 13, 2019
fanquake commented at 5:52 AM on September 13, 2019: member

If you are going to open PRs like this, please at least put the justification / reasoning in the PR description, use a properly formatted and explanatory commit message and ensure that the change is completely correct before opening.
GChuf commented at 11:25 AM on September 14, 2019: contributor

I apologise - I wrongly assumed the icons are of the same size. As for the reasoning, I thought it was obvious that duplicate files are not needed.

Would it be okay to set the project logo to bitcoin32.png and update PR description/commit message?
DrahtBot locked this on Dec 16, 2021

Contributors

Labels