trading-peter / chart-elements

Chart.js as Polymer Elements
https://robdodson.github.io/chart-elements
267 stars 70 forks source link

Transfer ownership #77

Closed robdodson closed 6 years ago

robdodson commented 6 years ago

Hey @pkaske I was curious if you'd like me to transfer ownership of this repo over to you? I think you've done a way better job of taking care of it than I ever have, and with the possible move to Lit in #76 it seems like a good time to hand things over. What do you think?

trading-peter commented 6 years ago

Sure! Makes me even proud to help one of the people who help drive the web platform forward to have more time to do so :)

I guess the only thing left to change after a transfer is to change the configuration for travis, right?

robdodson commented 6 years ago

I think so. Let me see if I can initiate the transfer now.

robdodson commented 6 years ago

oh it looks like you may need to delete your fork first.

image

trading-peter commented 6 years ago

I deleted my fork. But one question. Won't the transfer break installation via bower for current users, because the package path changes? If so, maybe it's better this repo should show a deprecation warning and redirect to a fork of mine?

robdodson commented 6 years ago

I think bower just hits GitHub which will redirect to the new owner. Similar to if we renamed the repo.

robdodson commented 6 years ago

Let's give it a shot. I can transfer to you and try a bower install. If it blows up then we can transfer back

trading-peter commented 6 years ago

Sounds good

trading-peter commented 6 years ago

I've accepted the transfer. It says it takes a few minutes.

... and it's already done, it seems

robdodson commented 6 years ago

My bower install seemed to work ok. I guess it's hard to know until a new tag is pushed

trading-peter commented 6 years ago

Hm yeah, seems like it works. Even after bower cache clean. But you're right, with the next tag it will probably show if everything really works for existing users.