hex-stickers icon indicating copy to clipboard operation
hex-stickers copied to clipboard

New logos for baguette, discrim, embed, rules

Open veeps opened this issue 4 years ago • 10 comments

  • [ x] I have converted fonts to outlines in the .svg
  • [ x] My PNG has dimensions 2206 x 2557

veeps avatar Aug 20 '20 01:08 veeps

Can you update from rstudio:master? There is a file or two missing.

topepo avatar Aug 24 '20 17:08 topepo

Just saw this notification... I did a pull upstream. Did that work?

veeps avatar Aug 29 '20 15:08 veeps

It didn't. The commit has

This commit does not belong to any branch on this repository.

At this point, the safest thing to do would be to save the new hex files somewhere, delete your repo, re-fork rstudio/hex-stickers, and do another PR.

topepo avatar Aug 29 '20 16:08 topepo

@topepo I'm not seeing what you're seeing; this branch looks fine to me.

hadley avatar Aug 30 '20 13:08 hadley

So it is ready to merge?

topepo avatar Aug 30 '20 14:08 topepo

Can we wait a week? We might be tweaking the size of the PNGs going forward

hadley avatar Aug 31 '20 12:08 hadley

No hurry

topepo avatar Aug 31 '20 14:08 topepo

/render

cderv avatar Nov 12 '21 12:11 cderv

/render

cderv avatar Nov 12 '21 12:11 cderv

Hi,

Just some context: I just rebuilt the README using the new GHA workflow that work for PR from fork. it was a way to test the workflow was working fine too.

cderv avatar Nov 12 '21 13:11 cderv

I'm going to close this PR momentarily because we're changing the default branch from master to main, but it would be great to get these finished off and into the repo. @veeps there's been two changes in the 2 years (eeek!) since you started this PR — we've changed the png size to 2521 x 2911 and we no longer include the url on the stickers (since (a) our name changed and (b) it was impossible to read on the stickers anyway). Do you have time to make those updates?

hadley avatar Nov 18 '22 14:11 hadley